[PATCH] D32911: [ScheduleDAG] Deal with already scheduled loads in ScheduleDAG.

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 06:56:07 PDT 2017


niravd added a comment.

Pinging again and adding comment to ease reviewer task.



================
Comment at: lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp:967
+    // this would negate the benefit to unfolding so just return SU.
+    if (LoadSU->isScheduled)
+      return SU;
----------------
For ease of review: these two line are the functional change. Everything else is just for cleanliness of code structure. 


https://reviews.llvm.org/D32911





More information about the llvm-commits mailing list