[PATCH] D90487: [CFG] Replace hardcoded max BBs explored as CL option. NFC.

[Anna Thomas via Phabricator via llvm-commits]

anna added inline comments.


================
Comment at: llvm/lib/Analysis/CFG.cpp:164
 
-  // Limit the number of blocks we visit. The goal is to avoid run-away compile
-  // times on large CFGs without hampering sensible code. Arbitrarily chosen.
-  unsigned Limit = 32;
+  unsigned Count = 0;
   SmallPtrSet<const BasicBlock*, 32> Visited;
----------------
jdoerfert wrote:
> If you initialize with DefaultMaxBBsToExplore you don't need to read the global multiple times.
good point, will do. Thanks for the review Johannes!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90487