[PATCH] D99444: [ADT] Introduce lazy unique iterator

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 27 02:32:37 PDT 2021


lebedev.ri marked an inline comment as done.
lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:310
+    if (!SI1Succs.count(Succ))
+      continue;
+    for (PHINode &PN : Succ->phis()) {
----------------
grandinj wrote:
> using make_unique_range here, but did not remove the old "make unique logic" ?
This is correct, we are dealing with two different ranges here,
and want to only deal with unique common successors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99444



More information about the llvm-commits mailing list