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