[PATCH] D43620: [Pipeliner] Fixed node order issue related to zero latency edges

Jan-Willem Roorda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 1 05:54:50 PDT 2020


jwroorda marked an inline comment as done.
jwroorda added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/MachinePipeliner.cpp:3908
+  typedef std::pair<SUnit *, unsigned> UnitIndex;
+  std::vector<UnitIndex> Indices(NodeOrder.size(), std::make_pair(nullptr, 0));
+
----------------
danilaml wrote:
> Why Indicies is initialized with NodeOrder.size() {nullptr, 0} pairs? Are they used anywhere?
The initialization is not needed. It can be removed.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D43620





More information about the llvm-commits mailing list