[PATCH] D126236: [CaptureTracking] Increase limit but use it for all visited uses.
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 23 13:30:35 PDT 2022
aeubanks added a comment.
this makes sense
is there a number you're seeing which roughly matches performance you were getting with typed pointers?
================
Comment at: llvm/lib/Analysis/CaptureTracking.cpp:452
// is captured to avoid taking too much compile time.
if (Count++ >= MaxUsesToExplore) {
Tracker->tooManyUses();
----------------
does moving this after `Visited.insert()` below help at all? I'd think that going through the worklist below is the expensive part, not constructing the worklist
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126236/new/
https://reviews.llvm.org/D126236
More information about the llvm-commits
mailing list