This is an archive of the discontinued LLVM Phabricator instance.

[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 Timeline

rjj created this revision.Feb 20 2023, 8:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2023, 8:02 AM
rjj requested review of this revision.Feb 20 2023, 8:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2023, 8:02 AM

About:

llvm/test/CodeGen/AArch64/neon-saba.ll

@dmgreen wrote in the other patch:

Nothing in this file seems to change. We already have abd tests for the nsw cases in neon-abd.ll.

These tests are for sabd, and so these tests are good additions. Is that it?

rjj added a comment.Feb 20 2023, 8:18 AM

About:

llvm/test/CodeGen/AArch64/neon-saba.ll

@dmgreen wrote in the other patch:

Nothing in this file seems to change. We already have abd tests for the nsw cases in neon-abd.ll.

These tests are for sabd, and so these tests are good additions. Is that it?

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.

dmgreen accepted this revision.Feb 20 2023, 9:05 AM

Yeah they sounds OK to add to me. The tests LGTM, whether you want to update the triple or not.

llvm/test/CodeGen/AArch64/neon-saba.ll
3

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.

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
This revision was automatically updated to reflect the committed changes.
rjj marked an inline comment as done.Feb 20 2023, 9:51 AM