<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
Good idea. Added in r350351 (<a href="https://reviews.llvm.org/rL350351" class="">https://reviews.llvm.org/rL350351</a>)
<div class=""><br class="">
</div>
<div class="">Artur<br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On 17 Dec 2018, at 20:55, Philip Reames <<a href="mailto:listmail@philipreames.com" class="">listmail@philipreames.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">Since the patch somewhat begs the question... test case?<br class="">
<br class="">
On 12/17/18 7:32 PM, Artur Pilipenko via llvm-commits wrote:<br class="">
<blockquote type="cite" class="">Author: apilipenko<br class="">
Date: Mon Dec 17 19:32:33 2018<br class="">
New Revision: 349438<br class="">
<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=349438&view=rev" class="">http://llvm.org/viewvc/llvm-project?rev=349438&view=rev</a><br class="">
Log:<br class="">
[CaptureTracking] Pass MaxUsesToExplore from wrappers to the actual implementation<br class="">
This is a follow up for rL347910. In the original patch I somehow forgot to pass<br class="">
the limit from wrappers to the function which actually does the job.<br class="">
<br class="">
Modified:<br class="">
llvm/trunk/lib/Analysis/CaptureTracking.cpp<br class="">
<br class="">
Modified: llvm/trunk/lib/Analysis/CaptureTracking.cpp<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/CaptureTracking.cpp?rev=349438&r1=349437&r2=349438&view=diff" class="">
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/CaptureTracking.cpp?rev=349438&r1=349437&r2=349438&view=diff</a><br class="">
==============================================================================<br class="">
--- llvm/trunk/lib/Analysis/CaptureTracking.cpp (original)<br class="">
+++ llvm/trunk/lib/Analysis/CaptureTracking.cpp Mon Dec 17 19:32:33 2018<br class="">
@@ -170,7 +170,7 @@ bool llvm::PointerMayBeCaptured(const Va<br class="">
(void)StoreCaptures;<br class="">
SimpleCaptureTracker SCT(ReturnCaptures);<br class="">
- PointerMayBeCaptured(V, &SCT);<br class="">
+ PointerMayBeCaptured(V, &SCT, MaxUsesToExplore);<br class="">
return SCT.Captured;<br class="">
}<br class="">
@@ -194,7 +194,8 @@ bool llvm::PointerMayBeCapturedBefore(co<br class="">
bool UseNewOBB = OBB == nullptr;<br class="">
if (!DT)<br class="">
- return PointerMayBeCaptured(V, ReturnCaptures, StoreCaptures);<br class="">
+ return PointerMayBeCaptured(V, ReturnCaptures, StoreCaptures,<br class="">
+ MaxUsesToExplore);<br class="">
if (UseNewOBB)<br class="">
OBB = new OrderedBasicBlock(I->getParent());<br class="">
@@ -202,7 +203,7 @@ bool llvm::PointerMayBeCapturedBefore(co<br class="">
// with StoreCaptures.<br class="">
CapturesBefore CB(ReturnCaptures, I, DT, IncludeI, OBB);<br class="">
- PointerMayBeCaptured(V, &CB);<br class="">
+ PointerMayBeCaptured(V, &CB, MaxUsesToExplore);<br class="">
if (UseNewOBB)<br class="">
delete OBB;<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@lists.llvm.org" class="">llvm-commits@lists.llvm.org</a><br class="">
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits<br class="">
</blockquote>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>