RFR (S) 8149813: Move trusted final field handling from C2 LoadNode::Value to shared code
aleksey.shipilev at oracle.com
Mon Feb 15 20:30:45 UTC 2016
Thank you for reviews, Vladimir and Tobias!
On 02/15/2016 10:06 PM, Vladimir Kozlov wrote:
> Yes, looks good.
> On 2/15/16 5:17 AM, Aleksey Shipilev wrote:
>> I would like us to consider moving out the "trusted fields" logic from
>> C2-ish LoadNode::Value to a shared code:
>> With this patch, primitive boxes and Strings are handled in the shared
>> code, where all other special cases are listed. Not only this is
>> cleaner, it also lets C1 to constant fold those cases too:
>> Some changes were needed in StringOpts, because it used to rely on
>> constant oop present in a graph after LoadNode::Value -- which was
>> inconsistent with the way we pull get_constant_coder anyway.
>> Testing: JPRT -testset hotspot, microbenchmarks, eyeballing assembly
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 819 bytes
Desc: OpenPGP digital signature
More information about the hotspot-compiler-dev