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

Tue Nov 19 16:46:08 UTC 2024

On Tue, 19 Nov 2024 14:51:47 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   use constant method handle to make the benchmark smaller
>
> Nice blog post, benchmarks, summary and explanation of the problem! The new heuristic looks reasonable and safer/simpler to use for JDK 24. Would be interesting to see if you could come up with a throughput and latency based heuristic/cost model at some point in the future.
> 
> I have some comments, mostly minor things.

@chhagedorn I think I addressed all your comments :)

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

PR Comment: https://git.openjdk.org/jdk/pull/21521#issuecomment-2486225491