[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