RFR: 8334431: C2 SuperWord: fix performance regression due to store-to-load-forwarding failures [v2]

[Emanuel Peter]

On Tue, 19 Nov 2024 13:54:21 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 25 commits:
>> 
>>  - manual merge
>>  - Merge branch 'master' into JDK-8334431-V-store-to-load-forwarding
>>  - Merge branch 'master' into JDK-8334431-V-store-to-load-forwarding
>>  - fix whitespace
>>  - fix tests and build
>>  - fix store-to-load forward IR rules
>>  - updates before the weekend ... who knows if they are any good
>>  - refactor to iteration threshold
>>  - use jvmArgs again, and apply same fix as 8343345
>>  - revert to jvmArgsPrepend
>>  - ... and 15 more: https://git.openjdk.org/jdk/compare/543e355b...000f9f13
>
> src/hotspot/share/opto/vtransform.cpp line 162:
> 
>> 160:   uint _memory_size;
>> 161:   bool _is_load;      // load or store?
>> 162:   uint _order;        // order in schedule
> 
> You could rename this to `_schedule_order`. Then you can remove the comment.

done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21521#discussion_r1848550596