[PATCH] D27585: [CaptureTracking] Add optimistic capture tracker for stores

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 22 15:38:04 PST 2016


hfinkel added a comment.

In https://reviews.llvm.org/D27585#630030, @sanjoy wrote:

> In https://reviews.llvm.org/D27585#623685, @JDevlieghere wrote:
>
> > Let's consider the following table:
>


...

> I think Hal's question was more of the tune of: do you have optimizations that will be correct even with false negatives?  That is, even if the optimization thinks a value did not escape when it actually did, the transform it does will be be correct?  If so, that is interestingly different from what LLVM does today, and worth discussion _before_ we add this new capture tracker to LLVM.

Yes, this is exactly what I'd like to know.


Repository:
  rL LLVM

https://reviews.llvm.org/D27585





More information about the llvm-commits mailing list