[PATCH] D87616: [ARM][LowOverheadLoops] Combine a VCMP and VPST into a VPT

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1326
+          // Register one
+          MIB.add(VCMP->getOperand(1));
+          // Register two
----------------
The VPT block pass has some very similar code. Do we need to check that Operand 1 and Operand 2 have not been modified between the VCMP and where we are materializing the VPT to?


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

https://reviews.llvm.org/D87616