[15] RFR(S) 8248822: 8 vm/classfmt/atr_ann/atr_rtm_annot007/atr_rtm_annot00709 tests fail w/ AOT

[Igor Veresov]

Thanks, Vladimir!

igor



> On Jul 8, 2020, at 3:16 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> I see that Doug and Tom approved these changes. I am fine with fix too.
> 
> Thanks,
> Vladimir
> 
> On 7/8/20 1:17 PM, Igor Veresov wrote:
>> The root cause of this is that Graal has intrinsics for jdk.internal.reflect.ConstantPool.{getIntAt0, getLongAt0, getFloatAt0, getDoubleAt0}() that don't check the range of the cp index or tag validity, whereas the original native implementations do. Since the utility of these intrinsics is of dubious value I’d like to remove it. The same change is going upstream as well.
>> Webrev: http://cr.openjdk.java.net/~iveresov/8248822/webrev.00/
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8248822
>> Thanks,
>> igor