RFR: 8310190: C2 SuperWord: AlignVector is broken, generates misaligned packs [v11]
Emanuel Peter
epeter at openjdk.org
Thu Nov 16 07:15:40 UTC 2023
On Wed, 15 Nov 2023 17:30:39 GMT, Boris Ulasevich <bulasevich at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 62 commits:
>>
>> - Merge branch 'master' into JDK-8311586
>> - fix flags register in VerifyAlignVector
>> - Merge branch 'master' into JDK-8311586
>> - small fix
>> - Merge branch 'master' into JDK-8311586
>> - add comments for must_verify_alignment
>> - fixed up verify_vector_alignment in ad file
>> - rename VerifyAlignment with VerifyVectorAlignment
>> - replace ifdef with Matcher::has_match_rule
>> - fix whitespace
>> - ... and 52 more: https://git.openjdk.org/jdk/compare/7c7f8ea3...6e7aed27
>
> Hi, on ARM32 jdk-tier1 and hotspot jtreg tests shows no regressing with this change.
@bulasevich thanks for running tier1! I fear that tier1 is not sufficient to test the auto-vectorizer / SuperWord. Would you mind rerunning tier1-3? And explicitly run the test `compiler/loopopts/superword/TestAlignVectorFuzzer.java` many times. @fg1417 hit some issues with that test.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14785#issuecomment-1813902079
More information about the hotspot-compiler-dev
mailing list