This option was hardcoded to 32. Changing this as a CL option since we
have seen some cases downstream where increasing this limit allows us to
disprove reachability.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM, one nit though
| llvm/lib/Analysis/CFG.cpp | ||
|---|---|---|
| 164 | If you initialize with DefaultMaxBBsToExplore you don't need to read the global multiple times. | |
| llvm/lib/Analysis/CFG.cpp | ||
|---|---|---|
| 164 | good point, will do. Thanks for the review Johannes! | |
If you initialize with DefaultMaxBBsToExplore you don't need to read the global multiple times.