RFR: 8275202: C2: optimize out more redundant conditions

[Roland Westrelin]

On Fri, 8 Dec 2023 14:53:34 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>>> I'm not. Out of curiosity, have you tried anything from Renaissance?
>> 
>> Thanks, I have not, will run some Renaissance benchmarks and report back.
>> 
>>> I will propose a new change where the slow down is not as big (the new pass is run less often).
>> 
>> Sounds good! I am happy to re-evaluate C2 speed when the new change is ready.
>
>> Thanks, I have not, will run some Renaissance benchmarks and report back.
> 
> Here are results from running a few Renaissance benchmarks on x64 and aarch64 (lower is better):
> 
> [score-baseline-vs-JDK-8275202-normalized-to-baseline.pdf](https://github.com/openjdk/jdk/files/13615435/score-baseline-vs-JDK-8275202-normalized-to-baseline.pdf)
> 
> To summarize, there seems to be a slight improvement on ScalaKmeans (x64 and aarch64) and a slight regression on Mnemonics/ParMnemonics (x64), the remaining differences are probably in the noise.

@robcasloz thanks!

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

PR Comment: https://git.openjdk.org/jdk/pull/14586#issuecomment-1847508317