RFR: 8310524: C2: record parser-generated LoadN nodes for IGVN [v2]

Daniel Lundén duke at openjdk.org
Wed Dec 6 08:50:40 UTC 2023


On Tue, 5 Dec 2023 16:03:28 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address comments
>
> test/hotspot/jtreg/compiler/c2/irTests/igvn/TestLoadNIdeal.java line 54:
> 
>> 52:         p[0] = new A();
>> 53: 
>> 54:         // Dummy is not compiled and hence not inlined => Escape analysis
> 
> Is there a reason you are not using [DontInline](https://github.com/openjdk/jdk/blob/master/test/hotspot/jtreg/compiler/lib/ir_framework/DontInline.java) to prevent inlining of `dummy`?

No particular reason. I checked, and `@DontInline` also makes the test work as expected. Is it preferable to use `@DontInline`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16967#discussion_r1416933860


More information about the hotspot-compiler-dev mailing list