[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