[PATCH] D132219: [SVE][LoopVectorize][NFC] Tidy up some tests
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 19 06:02:37 PDT 2022
david-arm created this revision.
david-arm added reviewers: sdesmalen, kmclaughlin, CarolineConcatto.
Herald added subscribers: shiva0217, ctetreau, psnobl, tschuett.
Herald added a reviewer: efriedma.
Herald added a project: All.
david-arm requested review of this revision.
Herald added subscribers: llvm-commits, alextsao1999.
Herald added a project: LLVM.
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!
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D132219
Files:
llvm/test/Transforms/LoopVectorize/AArch64/scalable-reductions.ll
llvm/test/Transforms/LoopVectorize/AArch64/sve-inductions-unusual-types.ll
llvm/test/Transforms/LoopVectorize/AArch64/sve-strict-fadd-cost.ll
llvm/test/Transforms/LoopVectorize/AArch64/sve-vector-reverse.ll
llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-gep.ll
llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-phi.ll
llvm/test/Transforms/LoopVectorize/AArch64/vector-reverse-mask4.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132219.453967.patch
Type: text/x-patch
Size: 41116 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220819/d2945f07/attachment.bin>
More information about the llvm-commits
mailing list