RFR(T) : 8250738 : C2Compiler::is_intrinsic_supported(methodHandle&,bool) shouldn't be virtual

Igor Ignatyev igor.ignatyev at oracle.com
Tue Jul 28 22:33:56 UTC 2020


Vladimir, Xin,

thank you for your reviews, pushed.

-- Igor

> On Jul 28, 2020, at 3:27 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> Good and trivial.
> 
> Thanks,
> Vladimir K
> 
> On 7/28/20 2:40 PM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8250738/webrev.00
>>> 2 lines changed: 0 ins; 0 del; 2 mod;
>> Hi all,
>> could you please review this trivial one-liner which removes virtual specifier from C2Compiler::is_intrinsic_supported(methodHandle&,bool)?
>> from JBS:
>>> C2Compiler::is_intrinsic_supported(methodHandle&,bool) is declared by C2Compiler which doesn't and shouldn't have any subclasses.
>> webrev: http://cr.openjdk.java.net/~iignatyev//8250738/webrev.00
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8250738
>> Thanks,
>> -- Igor



More information about the hotspot-compiler-dev mailing list