[PATCH] D26685: [Analysis] Add Basic Escape Analysis
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 2 05:57:47 PST 2016
hfinkel added a comment.
In https://reviews.llvm.org/D26685#611617, @JDevlieghere wrote:
> Hal, Philip, thanks for having a look. Because you both mentioned `CaptureTracking.cpp`, I took another look at the source code and I'm starting to doubt my original premise that there are cases where we have something that escapes but isn't captured. I'll do some testing over the weekend to make sure.
I'd also be interested in the opposite situations: Are there cases where things are captured but don't escape and that's uncovered by your analysis. I'm under the impressions that some of our capturing checks are really escape checks and more precision could be useful.
Repository:
rL LLVM
https://reviews.llvm.org/D26685
More information about the llvm-commits
mailing list