RFR: 8265711: C1: Intrinsify Class.getModifier method [v3]

Yi Yang yyang at openjdk.java.net
Mon May 17 02:20:38 UTC 2021


On Fri, 14 May 2021 16:11:22 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Yi Yang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   rename; redundant reloading
>
> test/hotspot/jtreg/compiler/c1/CanonicalizeGetModifiers.java line 31:
> 
>> 29:  * @requires vm.compiler1.enabled
>> 30:  * @library /test/lib
>> 31:  * @run main/othervm -XX:TieredStopAtLevel=1 -Xbatch
> 
> I would suggest to add 2 additional `@run` command to make sure test passed in all modes:
>  - default: without `-XX:TieredStopAtLevel`
>  - C2 only: with `-XX:-TieredCompilation`

Thank you Vladimir for taking time to look at this. Good suggestion! I've added.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3616


More information about the hotspot-compiler-dev mailing list