[PATCH] D90935: [ARM][LowOverheadLoops] Merge VCMP and VPST across VPT blocks

[Sam Tebbs via Phabricator via llvm-commits]

samtebbs added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1611
+            for (auto Op : I->operands()) {
+              if (Op.isReg() && !Op.isDef() && Op.getReg() == ARM::VPR) {
+                IntermediateInstrsUseVPR = true;
----------------
samparker wrote:
> This could be cleaned up by using std::any_of to iterate through the instructions and then also using the hasVPRUse helper too.
Oh nice, I didn't know these existed.


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

https://reviews.llvm.org/D90935