RFR(S): 8042997: Make intrinsic some or all check index/range methods
roland.westrelin at oracle.com
Mon Nov 16 09:23:05 UTC 2015
Thanks for the reviews Aleksey, Paul and Vladimir.
I removed the InlineObjectsCheckIndex flag and turned the jdk comment into an @implNote.
> On Nov 9, 2015, at 1:19 PM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
> Hotspot part looks good.
> I second Aleksey's suggestion to get rid of InlineObjectsCheckIndex flag.
> Best regards,
> Vladimir Ivanov
> On 11/9/15 2:24 PM, Roland Westrelin wrote:
>> This intrinsifies Objects.checkIndex() as a CmpU + RangeCheck node sequence that C2 then optimizes using the existing range check optimizations. The goal is allow library code to have range checks that are guaranteed to be properly optimized.
>> I’m not sure if we want to leave the comment on c2 in the library code.
More information about the hotspot-compiler-dev