8369881: C2: Unexpected node in SuperWord truncation: ReverseBytesS, ReverseBytesUS#233
8369881: C2: Unexpected node in SuperWord truncation: ReverseBytesS, ReverseBytesUS#233vish-chan wants to merge 1 commit intoopenjdk:masterfrom
Conversation
|
👋 Welcome back vish-chan! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
This backport pull request has now been updated with issue from the original commit. |
|
|
|
/approval request for Clean backport of 8369881: C2: Unexpected node in SuperWord truncation: ReverseBytesS, ReverseBytesUS This is a low risk fix adding 2 missing node types to Superword non-truncation list. Fixes debug assert failure with no impact in product. |
|
@vish-chan |
Clean backport of 8369881: C2: Unexpected node in SuperWord truncation: ReverseBytesS, ReverseBytesUS
Fixes fuzzer-discovered assert failure in SuperWord truncation for ReverseBytesS and ReverseBytesUS nodes. Adds missing nodes to truncation switch statement and includes test cases for both.
Additional Testing:
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk25u-dev.git pull/233/head:pull/233$ git checkout pull/233Update a local copy of the PR:
$ git checkout pull/233$ git pull https://git.openjdk.org/jdk25u-dev.git pull/233/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 233View PR using the GUI difftool:
$ git pr show -t 233Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk25u-dev/pull/233.diff
Using Webrev
Link to Webrev Comment