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

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: include/llvm/Analysis/AliasAnalysis.h:1100-1102
+/// AAResults does not invalidate the AAManager by default. It only needs to
+/// invalidate if it finds dependencies that are not preserved. Note that
+/// analyses should no longer be registered once the AAManager is run.
----------------
This would be a bit easier for me to read as:

```
The result of this analysis are only invalidated if one of the
particular aggregated AA results end up being invalidated. This
removes the need to explicitly preserve the results of `AAManager`.
Note that analyses should no longer be registered once the
`AAManager` is run.
```




Repository:
  rL LLVM

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

https://reviews.llvm.org/D60914