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

[Emanuel Peter]

On Tue, 19 Nov 2024 15:34:01 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   more for Christian
>
> test/micro/org/openjdk/bench/vm/compiler/VectorStoreToLoadForwarding.java line 91:
> 
>> 89:     }
>> 90: 
>> 91:     @CompilerControl(CompilerControl.Mode.DONT_INLINE)
> 
> Err are you sure this works I think this should be `FORCE_INLINE` instead. I see you want to have different `SIZE`s, too. Then you can make a `MutableCallSite` for each parameter. The magic here is that the compiler will treat the call target as a constant and force a recompilation each time you call `setTarget`

Hmm, somehow it looked like it worked... but it cannot with this. I'll have another look.

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

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