[PATCH] D18093: [ScheduleDAGInstrs] Handle instructions with multiple MMOs

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 08:38:40 PDT 2016


mcrosier added a comment.

Seems reasonable to me, but I think Hal/Andy or someone else more familiar with this could should approve the patch.


================
Comment at: lib/CodeGen/ScheduleDAGInstrs.cpp:1048
@@ +1047,3 @@
+      // self-loop edge if multiple underlying objects are present.
+      for (auto &underlObj : Objs) {
+        ValueType V = underlObj.getPointer();
----------------
underlObj should be capitalized.


http://reviews.llvm.org/D18093





More information about the llvm-commits mailing list