[PATCH] D110368: [AA] Move earliest escape tracking from DSE to AA

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 24 12:52:27 PDT 2021


nikic added a comment.

In D110368#3020035 <https://reviews.llvm.org/D110368#3020035>, @fhahn wrote:

>> This combines the "earliest escape" analysis with the full power of BasicAA:
>
> Are there any test cases that show the benefit of that, other than the improvements for the load-load case discussed in D109844 <https://reviews.llvm.org/D109844> ?

See the last three tests in captures-before-load.ll. Only the last one truly requires BasicAA integration though.

Ultimately my main interest here is to integrate this functionality in a way that does not require reimplementing parts of AA in DSE. Better analysis quality is just a nice side effect.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110368/new/

https://reviews.llvm.org/D110368



More information about the llvm-commits mailing list