[PATCH] D63282: [MachinePipeliner] Don't check boundary node in checkValidNodeOrder

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 13:14:19 PDT 2019


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

LGTM



================
Comment at: llvm/lib/CodeGen/MachinePipeliner.cpp:3736
+      // Do not process a boundary node, it was not included in NodeOrder,
+      // hence not in Indices either,std::lower_bound() will return
+      // Indices.end() below.
----------------
space after , before std.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63282/new/

https://reviews.llvm.org/D63282





More information about the llvm-commits mailing list