RFR: 8316594: C2 SuperWord: wrong result with hand unrolled loops [v3]

Emanuel Peter epeter at openjdk.org
Wed Oct 4 13:07:18 UTC 2023


> Please read description in https://github.com/openjdk/jdk/pull/15864.
> 
> Currently, we only compare the first element of the load-pack with the StoreVector. This is not correct, we need to compare all elements of the load-pack with the StoreVector. Otherwise it may be that the first element lays below/before the memory region of the StoreVector (Less), but the last element of the load-pack for example could have overlap with the StoreVector (Equal).
> 
> Tier1-6 + stress-testing.
> Performance testing.

Emanuel Peter has updated the pull request incrementally with two additional commits since the last revision:

 - rm whitespaces
 - patched test

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15866/files
  - new: https://git.openjdk.org/jdk/pull/15866/files/7225f734..1b6b0582

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15866&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15866&range=01-02

  Stats: 194 lines in 2 files changed: 87 ins; 106 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/15866.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15866/head:pull/15866

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


More information about the hotspot-compiler-dev mailing list