[PATCH] D69845: [ARM][MVE] canTailPredicateLoop

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 8 06:34:35 PST 2019


SjoerdMeijer updated this revision to Diff 228433.
SjoerdMeijer added a comment.

Thanks for checking. Got myself confused about the loads/stores, but I think we indeed want to accept extending loads and narrowing stores, so I've added checks and a bunch of tests:

- for SEXT/ZEXT, the operand needs to be a load,
- for TRUNC, the user needs to be a store.

And have also allowed the float types.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69845/new/

https://reviews.llvm.org/D69845

Files:
  llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
  llvm/test/Transforms/LoopVectorize/ARM/prefer-tail-loop-folding.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69845.228433.patch
Type: text/x-patch
Size: 29087 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191108/dfce5e87/attachment.bin>


More information about the llvm-commits mailing list