This is an archive of the discontinued LLVM Phabricator instance.

[SVE][LoopVectorize][NFC] Tidy up some tests
ClosedPublic

Authored by david-arm on Aug 19 2022, 6:02 AM.

Details

Summary

Whilst writing a patch to add extra tail-folding RUN lines to
existing tests I noticed a few areas where they can be
cleaned up a little:

  1. scalable-reductions.ll: fmin_fast does not mark fcmp as fast.
  2. sve-inductions-unusual-types.ll: remove direct references to SSA variable names.
  3. sve-strict-fadd-cost.ll: don't force vector width so we see costs for different VFs in one go. This will be important for the follow-on patch.
  4. sve-vector-reverse.ll,vector-reverse-mask4.ll: add noalias keyword to simplify IR.
  5. sve-widen-gep.ll,sve-widen-phi.ll: regenerate using script.

These changes will make the subsequent patch adding RUN lines much
easier to review!

Diff Detail

Event Timeline

david-arm created this revision.Aug 19 2022, 6:02 AM
Herald added a project: Restricted Project. · View Herald Transcript
david-arm requested review of this revision.Aug 19 2022, 6:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 19 2022, 6:02 AM
sdesmalen accepted this revision.Aug 19 2022, 6:25 AM
This revision is now accepted and ready to land.Aug 19 2022, 6:25 AM
This revision was automatically updated to reflect the committed changes.