[PATCH] D48666: [ScheduleDAG] Fix unfolding of SUnits to already existent nodes.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 12:28:34 PDT 2018


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM with one nit



================
Comment at: llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp:1013
 
+  bool isNewN = true;
+  SUnit *NewSU;
----------------
Please add a comment here noting that we specifically end up in this situation when isNewLoad == false.


Repository:
  rL LLVM

https://reviews.llvm.org/D48666





More information about the llvm-commits mailing list