RFR: 8361353: [PPC64] C2: Add nodes UMulHiL, CmpUL3, UMinV, UMaxV, NegVI [v2]

Martin Doerr mdoerr at openjdk.org
Fri Jul 4 10:11:39 UTC 2025


On Fri, 4 Jul 2025 08:16:59 GMT, David Briemann <dbriemann at openjdk.org> wrote:

>> Implement more nodes for ppc that exist on other platforms.
>
> David Briemann has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add >= power9 check for NegVI

I suggest removing the NegVI again.

test/hotspot/jtreg/compiler/intrinsics/TestCompareUnsigned.java line 34:

> 32:  * @bug 8283726 8287925
> 33:  * @requires os.arch=="amd64" | os.arch=="x86_64" | os.arch=="aarch64" | os.arch=="riscv64" | os.arch=="ppc64" | os.arch=="ppc64le"
> 34: 

The test expects "CmpU3" for integers to be available. Can you implement that, too, please?

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

Changes requested by mdoerr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26115#pullrequestreview-2986428202
PR Review Comment: https://git.openjdk.org/jdk/pull/26115#discussion_r2184925416


More information about the hotspot-compiler-dev mailing list