This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SME]: Add streaming-compatible testing files.(frame-offsets.ll, int-immediates.ll, log-reduce.ll)
ClosedPublic

Authored by hassnaa-arm on Nov 25 2022, 7:31 AM.

Details

Summary

Testing files:

  • frame-offsets.ll
  • int-immediates.ll
  • log-reduce.ll

Diff Detail

Event Timeline

hassnaa-arm created this revision.Nov 25 2022, 7:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 25 2022, 7:31 AM
hassnaa-arm requested review of this revision.Nov 25 2022, 7:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 25 2022, 7:31 AM
Matt added a subscriber: Matt.Nov 27 2022, 10:36 AM

Add int-compares.ll testing file taht doesn't need any change.

This revision is now accepted and ready to land.Nov 29 2022, 8:32 AM
hassnaa-arm retitled this revision from [AArch64][SME]: Add streaming-compatible testing files. to [AArch64][SME]: Add streaming-compatible testing files.(frame-offsets.ll, int-immediates.ll, log-reduce.ll).Nov 30 2022, 1:23 AM
sdesmalen added inline comments.Nov 30 2022, 2:45 AM
llvm/test/CodeGen/AArch64/sve-streaming-mode-fixed-length-frame-offsets.ll
8

I don't see the value of this test, given that this test uses no SVE loads/stores. I think you can just remove the whole file?

llvm/test/CodeGen/AArch64/sve-streaming-mode-fixed-length-int-immediates.ll
5

This change needs a FIXME, because none of the instructions use the immediate form.

14

Please use opaque pointers for all tests.

hassnaa-arm marked 3 inline comments as done.

Remove not needed frame-offsets.ll file
Use opaque PTRs

sdesmalen accepted this revision.Nov 30 2022, 6:30 AM