RFR: 8345609: [C1] LIR Operations with one input should be implemented as LIR_Op1
Richard Reingruber
rrich at openjdk.org
Thu Dec 5 21:20:37 UTC 2024
On Thu, 5 Dec 2024 16:57:24 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.
Nice cleanup 👍
A few files need copyright update // especially c1_LinearScan_x86.hpp ;)
Cheers, Richard.
-------------
Marked as reviewed by rrich (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22582#pullrequestreview-2482983552
More information about the hotspot-compiler-dev
mailing list