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

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 17 02:35:50 PST 2020


samparker 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;
----------------
This could be cleaned up by using std::any_of to iterate through the instructions and then also using the hasVPRUse helper too.


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

https://reviews.llvm.org/D90935



More information about the llvm-commits mailing list