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