[PATCH] D52575: [SimplifyCFG] Pass AggressiveInsts to DominatesMergePoint by reference. Remove null check.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 4 16:06:34 PDT 2018


craig.topper added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:353
   // If we have seen this instruction before, don't count it again.
-  if (AggressiveInsts->count(I))
+  if (AggressiveInsts.count(I))
     return true;
----------------
xbolva00 wrote:
> AggressiveInsts.find(I) != AggressiveInsts.end()
Why?


https://reviews.llvm.org/D52575





More information about the llvm-commits mailing list