Value type array optimizations/fixes with C2

Tobias Hartmann tobias.hartmann at oracle.com
Tue Feb 14 09:12:46 UTC 2017


Hi,

On 14.02.2017 10:10, David Simms wrote:
> I can't answer for the JIT parts, but the rest of it looks good !

Thanks for looking at this!

Best regards,
Tobias

> On 13/02/17 17:39, Tobias Hartmann wrote:
>> Hi,
>>
>> http://cr.openjdk.java.net/~thartmann/valhalla/vt_prototype/webrev.07/
>>
>> please review this change which includes:
>> - Fixed wrong return value in AddPNode::Value()
>> - Basic version of ValueArrayKlass::oop_print_on() for debugging
>> - Fixed escape analysis to keep track of the field being accessed in VT arrays
>> - Re-enabled asserts in graphKit.cpp and memnode.cpp
>> - Implemented scalar replacement for flattened value type arrays
>> - Implemented re-allocation of scalar replaced value type arrays on deoptimization
>> - Refactored valuetypenode.cpp and related code
>> - Refactored test bench to use flags from jtreg command line instead of hard-coding command line options
>> - Enabled/added corresponding regression tests and match rules
>>
>> There are still some problems with incremental inlining. I disabled the flag for testing and will work on fixing this next.
>>
>> Thanks,
>> Tobias
> 
> 


More information about the valhalla-dev mailing list