Integrated: 8370579: PPC: fix inswri immediate argument order

Manuel Hässig mhaessig at openjdk.org
Tue Oct 28 09:02:45 UTC 2025


On Fri, 24 Oct 2025 15:19:06 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

> This cleanup PR swaps and renames the immediate arguments of the two `insrwi` instruction macros in `ppc.ad` such that they correspond to the order and names in the manual. This involved swapping the arguments in all six usages. I hope this saves the next person trying to reason about this some confused hours.
> 
> Testing:
>  - [x] Github Actions
>  - [x] Running some relevant tests (`compiler/c2/TestCharShortByteSwap.java`, `jdk/java/lang/Short/ByteSwap.java`, `jdk/java/lang/Integer/BitTwiddle.java`, `compiler/codegen/Test6431242.java`) in qemu

This pull request has now been integrated.

Changeset: 96259936
Author:    Manuel Hässig <mhaessig at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9625993611bb6acf84d428bea4a65d33b9d66e5f
Stats:     13 lines in 1 file changed: 0 ins; 0 del; 13 mod

8370579: PPC: fix inswri immediate argument order

Reviewed-by: mdoerr, mbaesken

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

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


More information about the hotspot-compiler-dev mailing list