RFR: 8345609: [C1] LIR Operations with one input should be implemented as LIR_Op1 [v2]

Richard Reingruber rrich at openjdk.org
Fri Dec 6 08:36:39 UTC 2024


On Thu, 5 Dec 2024 21:29:51 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> Change `lir_sqrt`, `lir_abs`, `lir_neg`, `lir_f2hf`, `lir_hf2f` to use `LIR_Op1`. Extend `LIR_Op1` to support one temp register operand.
>> Also see JBS issue.
>> 
>> Remove `lir_tan` and `lir_log10` which are unused (dead and incomplete code). They should have been removed with or after https://github.com/openjdk/jdk/commit/ad79a5ae65d24861ead3ae96cf148c8bc0f02736 and the corresponding changes on other platforms.
>
> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update Copyright headers.

You might even see an effect in the `Fp16ConversionBenchmark` if you stop compilation at tier 1.

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

PR Comment: https://git.openjdk.org/jdk/pull/22582#issuecomment-2522508261


More information about the hotspot-compiler-dev mailing list