[PATCH] D67709: [ARM][MVE] Cleanup tail-predicated loop

Thu Sep 19 06:04:22 PDT 2019

samparker updated this revision to Diff 220848.
samparker added a comment.

Added 'hasSideEffects' to VCTP because machine cse did a good job of removing the duplicate vctp in the exit block. I think always executing the 'extra' instruction outweighs the risk of having to spill/reload the vpr or not being able to perform the proper tail predication.

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

https://reviews.llvm.org/D67709

Files:
  lib/Target/ARM/ARMInstrMVE.td
  lib/Target/ARM/MVETailPredication.cpp
  test/CodeGen/Thumb2/LowOverheadLoops/vector-reduce-mve-codegen.ll
  test/CodeGen/Thumb2/LowOverheadLoops/vector-reduce-mve-tail.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67709.220848.patch
Type: text/x-patch
Size: 10484 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190919/94a07351/attachment.bin>