[PATCH] D68191: Simplify function llvm::removeUnreachableBlocks() to avoid (re-)computation.

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:2232
+    //skip reachable basic blocks
+    if (Reachable.find(BB)!=Reachable.end())
       continue;
----------------
xbolva00 wrote:
> Since Reachable is SmallPtrSet, count should be as good as find()
> 
> https://github.com/llvm-mirror/llvm/blob/master/include/llvm/ADT/SmallPtrSet.h#L381
Please “clang format” this patch


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68191