[PATCH] D90461: [ARM][LowOverheadLoops] Merge a VCMP and the new VPST into a VPT

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1571
+            RDA->getUniqueReachingMIDef(VPST, ARM::VPR) == VCMP) {
+          ReplaceVCMPWithVPT();
+        } else if (DivergentNextIsPredicated) {
----------------
I'm still not sure this is the correct VPST to be looking at, in order to replace a VCMP with a VPT. I think either that this will be the last instruction in the block, in which case it needs to be folded into the _next_ instruction, or it will be in the middle of a block that is handled below.

Oh, whilst I write this I see you have a patch for that, fantastic. Is this if block needed, or can it be removed without altering anything else?


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

https://reviews.llvm.org/D90461