diff options
Diffstat (limited to 'llvm/lib/Analysis')
-rw-r--r-- | llvm/lib/Analysis/BasicAliasAnalysis.cpp | 13 | ||||
-rw-r--r-- | llvm/lib/Analysis/CaptureTracking.cpp | 2 |
2 files changed, 14 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/BasicAliasAnalysis.cpp b/llvm/lib/Analysis/BasicAliasAnalysis.cpp index aaa79bc0199..55a53f2122a 100644 --- a/llvm/lib/Analysis/BasicAliasAnalysis.cpp +++ b/llvm/lib/Analysis/BasicAliasAnalysis.cpp @@ -132,7 +132,18 @@ static bool isNonEscapingLocalObject(const Value *V) { /// Returns true if the pointer is one which would have been considered an /// escape by isNonEscapingLocalObject. static bool isEscapeSource(const Value *V) { - if (isa<CallInst>(V) || isa<InvokeInst>(V) || isa<Argument>(V)) + if (auto CS = ImmutableCallSite(V)) { + // launder_invariant_group captures its argument only by returning it, + // so it might not be considered an escape by isNonEscapingLocalObject. + // Note that adding similar special cases for intrinsics in CaptureTracking + // requires handling them here too. + if (CS.getIntrinsicID() == Intrinsic::launder_invariant_group) + return false; + + return true; + } + + if (isa<Argument>(V)) return true; // The load case works because isNonEscapingLocalObject considers all diff --git a/llvm/lib/Analysis/CaptureTracking.cpp b/llvm/lib/Analysis/CaptureTracking.cpp index 782e277222f..54466cd3467 100644 --- a/llvm/lib/Analysis/CaptureTracking.cpp +++ b/llvm/lib/Analysis/CaptureTracking.cpp @@ -249,6 +249,8 @@ void llvm::PointerMayBeCaptured(const Value *V, CaptureTracker *Tracker) { // launder.invariant.group only captures pointer by returning it, // so the pointer wasn't captured if returned pointer is not captured. + // Note that adding similar special cases for intrinsics requires handling + // them in 'isEscapeSource' in BasicAA. if (CS.getIntrinsicID() == Intrinsic::launder_invariant_group) { AddUses(I); break; |