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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Can someone look at D83612 <https://reviews.llvm.org/D83612>, D85168 <https://reviews.llvm.org/D85168>, D92781 <https://reviews.llvm.org/D92781>, D83607 <https://reviews.llvm.org/D83607> and D83613 <https://reviews.llvm.org/D83613>? We seem to have the base parts for new PM codegen, but they've been languishing in review waiting. We're stuck waiting for these with no direction



================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1091
+  Pass *LegacyPass = nullptr;
 };
 
----------------
jdoerfert wrote:
> arsenm wrote:
> > jdoerfert wrote:
> > > arsenm wrote:
> > > > aeubanks wrote:
> > > > > arsenm wrote:
> > > > > > 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
> > > > > is it possible to run whatever pass you're interested in as part of the optimization pipeline?
> > > > Not in the right place. We can't even move all the IR passes because of the one RegionPass which new PM doesn't have.
> > > Shouldn't we invest the time and code to provide a "region pass wrapper"? We stopped testing most of these things with the old PM, using it is not a great idea.
> > We should invest the time in killing of the structurizer entirely. It being a region pass is a problem for many reasons. But that's hard
> @arsenm Where do I find that structurizer and what should the replacement do/look like? It's not https://github.com/llvm/llvm-project/blob/main/llvm/lib/Transforms/Scalar/StructurizeCFG.cpp right?
> 
That is the one.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138991