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

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


mcrosier added inline comments.

================
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();
----------------
gberry wrote:
> mcrosier wrote:
> > underlObj should be capitalized.
> This is a copy of the loop above.  I think the capitalization of this (and stores_, which should probably also drop the '_') should be a separate change.
Fair enough.


http://reviews.llvm.org/D18093





More information about the llvm-commits mailing list