[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