Split off the tests from D144379.
This adds tests for saba. SVE tests currently exhibit inefficient codegen.
Paths
| Differential D144399
[AArch64] Add tests for saba (NFC) ClosedPublic Authored by rjj on Feb 20 2023, 8:02 AM.
Details
Summary Split off the tests from D144379. This adds tests for saba. SVE tests currently exhibit inefficient codegen.
Diff Detail
Event TimelineComment Actions About: llvm/test/CodeGen/AArch64/neon-saba.ll @dmgreen wrote in the other patch:
These tests are for sabd, and so these tests are good additions. Is that it? Comment Actions
These tests are for saba. It is true they don't change (i.e. the current code generated is good), I just thought to include them for completeness. But I can drop them if you think they are unnecessary. Comment Actions Yeah they sounds OK to add to me. The tests LGTM, whether you want to update the triple or not.
This revision is now accepted and ready to land.Feb 20 2023, 9:05 AM This revision was landed with ongoing or failed builds.Feb 20 2023, 9:44 AM Closed by commit rG4077e8ab2f09: [AArch64] Add tests for saba (NFC) (authored by rjj). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 498896 llvm/test/CodeGen/AArch64/neon-saba.ll
llvm/test/CodeGen/AArch64/sve-saba.ll
|
I think most tests use -mtriple aarch64-unknown-linux-gnu -mattr=+neon.
But I think +neon is default, so shouldn't be needed. For the sve tests it will be needed though.