[PATCH] D97225: Only verify LazyCallGraph under expensive checks

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 13:39:20 PST 2021


aeubanks created this revision.
Herald added a subscriber: hiraditya.
aeubanks requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

These verify calls are causing a lot of slowdown on some files, up to 8x.
The LazyCallGraph infra has been tested a lot over the years, so I'm fairly confident that we don't always need to run the verifys.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D97225

Files:
  llvm/lib/Analysis/CGSCCPassManager.cpp
  llvm/lib/Analysis/LazyCallGraph.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97225.325556.patch
Type: text/x-patch
Size: 9607 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210222/e281b957/attachment.bin>


More information about the llvm-commits mailing list