[PATCH] D60914: [AliasAnalysis] AAResults preserves AAManager.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 19 15:10:03 PDT 2019


chandlerc added a comment.

In D60914#1473167 <https://reviews.llvm.org/D60914#1473167>, @asbirlea wrote:

> > You should also update the comments on AAManager to explain this, and further add the (largely already in place) restriction that the registration functions must not be called once the AAManager is run.
>
> Could you please elaborate on this? How/where is this restriction enforced?


It's not enforced at all, that's why we should document it.... We could add enforcement too, but it is a difficult mistake to make, and so I'm not worried about it. (Explained in person the nature of the "issue".)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60914





More information about the llvm-commits mailing list