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

[Andrew Trick via llvm-commits]

atrick 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();
----------------
MatzeB wrote:
> This will subregister defs which are uses as well, should be `!MO.readsReg())!
Matthias, can you clarify? We don't want addPhysRefDefs to be called twice for the same subreg!


Repository:
  rL LLVM

http://reviews.llvm.org/D20102