[PATCH] D88093: [ARM][MVE] Enable tail-predication by default

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer created this revision.
SjoerdMeijer added reviewers: efriedma, samparker, dmgreen, samtebbs.
Herald added subscribers: danielkiss, hiraditya, kristof.beyls.
Herald added a project: LLVM.
SjoerdMeijer requested review of this revision.

We have been running tests/benchmarks downstream with tail-predication enabled for some time now and this behaves as expected: we are not aware of any correctness issues, and this performs better across the board than disabling tail-predication.

So, if we can get D88086 <https://reviews.llvm.org/D88086> out of the way, I think it is time to flip the switch.


https://reviews.llvm.org/D88093

Files:
  llvm/lib/Target/ARM/MVETailPredication.cpp
  llvm/test/Transforms/LoopVectorize/ARM/mve-reduction-types.ll
  llvm/test/Transforms/LoopVectorize/ARM/pointer_iv.ll
  llvm/test/Transforms/LoopVectorize/ARM/tail-folding-reduces-vf.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88093.293440.patch
Type: text/x-patch
Size: 71140 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200922/f8c8cbc7/attachment-0001.bin>