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