[PATCH] D138991: [AAPointerInfo] handle multiple offsets in PHI

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1091
+  Pass *LegacyPass = nullptr;
 };
 
----------------
jdoerfert wrote:
> Do we really need legacy PM support? I would prefer not to add complexity for it.
Yes. Contrary to popular belief, the new PM is not complete. Until we have codegen support, we need legacy PM support for IR passes run in the backend


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138991