[PATCH] D100435: [ARM] Transforming memset to Tail predicated Loop

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:11279
+    // fallthrough is the exit.
+    TpExit = BB->splitAt(MI);
     if (TpExit == BB) {
----------------
Why has this changed from false to true?

What happens if CPSR is live across the MVE_MEMCPYLOOPINST?  WhileLoopStart and LoopEnd both clobber that physical register, incase they get reverted to subs; bne. Do we need to add the same clobber to these new MVE mem loop instructions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100435