[PATCH] D41203: [PM][InstCombine] fixing omission of AliasAnalysis in new-pass-manager's version of InstCombine

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 21:27:19 PST 2017


skatkov added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:3289
   PA.preserve<AAManager>();
+  PA.preserve<BasicAA>();
   PA.preserve<GlobalsAA>();
----------------
Chandler, may be you can answer this question: why do we need to preserve BasicAA if we already preserved AAManager?


https://reviews.llvm.org/D41203





More information about the llvm-commits mailing list