[PATCH] D71348: Add ExternalAAWrapperPass to createLegacyPMAAResults.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 05:49:11 PST 2019


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

In D71348#1783455 <https://reviews.llvm.org/D71348#1783455>, @nhaehnle wrote:

> Thanks for the heads-up. I think this is perfectly fine from an AMDGPU perspective. It also seems like a simple oversight that the ExternalAAWrapperPass is not in that list, so this LGTM in general, but I'd wait a bit for somebody who is more familiar with alias analysis to weigh in.


LGTM too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71348





More information about the llvm-commits mailing list