[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