8376803: Jtreg test compiler/vectorization/TestVectorAlgorithms.java fails after JDK-8373026#31375
8376803: Jtreg test compiler/vectorization/TestVectorAlgorithms.java fails after JDK-8373026#31375Sorna-Sarathi wants to merge 1 commit into
Conversation
|
👋 Welcome back ssarathi! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
@Sorna-Sarathi The following label will be automatically applied to this pull request:
When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing list. If you would like to change these labels, use the /label pull request command. |
|
The total number of required reviews for this PR has been set to 2 based on the presence of this label: |
… after JDK-8373026
7187028 to
27dc977
Compare
|
@Sorna-Sarathi Please do not rebase or force-push to an active PR as it invalidates existing review comments. Note for future reference, the bots always squash all changes into a single commit automatically as part of the integration. See OpenJDK Developers’ Guide for more information. |
JDK-8373026 introduced TestVectorAlgorithms. The vectorized hash computations implicitly assumed little-endian lane ordering, causing the test to fail on big-endian platforms.
This fix updates the hash computation to preserve logical byte order across platforms, making the test endianness-independent and ensuring consistent behaviour on both little-endian and big-endian architectures.
JBS Issue: JDK-8376803
Progress
Error
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/31375/head:pull/31375$ git checkout pull/31375Update a local copy of the PR:
$ git checkout pull/31375$ git pull https://git.openjdk.org/jdk.git pull/31375/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 31375View PR using the GUI difftool:
$ git pr show -t 31375Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/31375.diff
Using Webrev
Link to Webrev Comment