RFR: 8351861: RISC-V: add simple assert at arrays_equals_v

Hamlin Li mli at openjdk.org
Wed Mar 12 14:53:03 UTC 2025


Hi,
Can you help to review this trivial patch?

`arrays_equals_v` and `arrays_equals` are 2 versions of the same node `AryEqNode`, input `elem_size` should be the same, so should share the same assert of `elem_size`, this also make the code below more clear.
Although at the same time, we could do the similar thing like https://github.com/openjdk/jdk/pull/24006, but as the code of `arrays_equals_v` and `arrays_equals` seems not require the input must be a byte[] (although in the java level a string's payload is indeed a byte[]), so I'll just leave if as is.

Thanks

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

Commit messages:
 - assert diff registers
 - initial commit

Changes: https://git.openjdk.org/jdk/pull/24008/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24008&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8351861
  Stats: 3 lines in 1 file changed: 3 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/24008.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24008/head:pull/24008

PR: https://git.openjdk.org/jdk/pull/24008


More information about the hotspot-compiler-dev mailing list