[PATCH] D55042: Introduce MaxUsesToExplore argument to capture tracking

Artur Pilipenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 28 18:34:29 PST 2018


apilipenko created this revision.
apilipenko added reviewers: hfinkel, sanjoy, majnemer, rnk.
Herald added a subscriber: llvm-commits.

Currently CaptureTracker gives up if it encounters a value with more than 20 uses. The motivation for this cap is to keep it relatively cheap for BasicAliasAnalysis use case, where the results can't be cached. Although, other clients of CaptureTracker might be ok with higher cost. This patch introduces an argument for PointerMayBeCaptured functions to specify the max number of uses to explore. The motivation for this change is a downstream user of CaptureTracker, but I believe upstream clients of CaptureTracker might also benefit from more fine grained cap.


Repository:
  rL LLVM

https://reviews.llvm.org/D55042

Files:
  include/llvm/Analysis/CaptureTracking.h
  lib/Analysis/CaptureTracking.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55042.175801.patch
Type: text/x-patch
Size: 5836 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181129/2b92d838/attachment.bin>


More information about the llvm-commits mailing list