[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