RFR: 8350480: RISC-V: Relax assertion about registers in C2_MacroAssembler::minmax_fp

Hamlin Li mli at openjdk.org
Fri Feb 21 10:35:51 UTC 2025


On Fri, 21 Feb 2025 06:36:52 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Hi, please review this trivial change.
> The current assertion about the registers is more than needed.
> It requires that `dst`, `src1` and `src2` must be different from each other.
> But the code only required that `dst` must be different from `src1` and `src2`.
> Patch simply relaxes the assersion removing the unneeded constraint.
> fastdebug builds OK with change.

Looks good.

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

Marked as reviewed by mli (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23723#pullrequestreview-2632648676


More information about the hotspot-compiler-dev mailing list