RFR: 8320649: C2: Optimize scoped values [v9]

Roland Westrelin roland at openjdk.org
Mon Mar 4 12:35:56 UTC 2024


On Mon, 26 Feb 2024 14:04:06 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   32 bit build fix
>
> src/hotspot/share/opto/callGenerator.cpp line 854:
> 
>> 852: 
>> 853:     // Pattern matches:
>> 854:     // if ((objects = scopedValueCache()) != null) {
> 
> Suggestion:
> 
>     // if (scopedValueCache() != null) {
> 
> You don't use `objects` here, so it just confused me.

I use snippets from the java code for `ScopedValue.get()` in the comments so it's easier to see what's being pattern matched.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16966#discussion_r1511095016


More information about the hotspot-compiler-dev mailing list