[PATCH] D20102: [ScheduleDAG] Make sure to process all def operands before any use operands

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 11:05:12 PDT 2016


MatzeB added inline comments.

================
Comment at: llvm/trunk/lib/CodeGen/ScheduleDAGInstrs.cpp:974-975
@@ +973,4 @@
+      const MachineOperand &MO = MI->getOperand(j);
+      if (!MO.isReg() || !MO.isUse())
+        continue;
+      unsigned Reg = MO.getReg();
----------------
This will subregister defs which are uses as well, should be `!MO.readsReg())!


Repository:
  rL LLVM

http://reviews.llvm.org/D20102





More information about the llvm-commits mailing list