This is an archive of the discontinued LLVM Phabricator instance.

[NewPM] Run avx*-builtins.c tests under the new pass manager only
ClosedPublic

Authored by leonardchan on Jul 22 2019, 1:26 PM.

Details

Summary

This patch changes the following tests to run under the new pass manager only:

Clang :: CodeGen/avx512-reduceMinMaxIntrin.c (1 of 4)
Clang :: CodeGen/avx512vl-builtins.c (2 of 4)
Clang :: CodeGen/avx512vlbw-builtins.c (3 of 4)
Clang :: CodeGen/avx512f-builtins.c (4 of 4)

The new PM added extra bitcasts that weren't checked before. For reduceMinMaxIntrin.c, the issue was mostly the alloca's being in a different order. Other changes involved extra bitcasts, and differently ordered loads and stores, but the logic should still be the same.

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon added inline comments.Jul 23 2019, 2:56 PM
clang/test/CodeGen/avx512vlbw-builtins.c
904 ↗(On Diff #211177)

Can you release the DSTTY/SRCTY patterns with the actual types to improve readability?

leonardchan marked an inline comment as done.
This revision is now accepted and ready to land.Jul 26 2019, 2:12 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 26 2019, 2:19 PM