[llvm] [AA] Move Target Specific AA before BasicAA (PR #125965)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu May 8 02:24:59 PDT 2025


arsenm wrote:

> And @arsenm commented that it may not be a good idea to have a boolean flag `Early` inside a wrapper pass. What would be a better way to implement this? What I can think of is to define a `virtual` function `runEarly()`, which by default will return `false`. And the backend can override this function to let it return `true`. This can make the change a little bit simpler, while this change is still similar to have a flag in the wrapper pass.

The virtual override is the same thing, but worse. I'd rather revert back to the flag if this is really configurable 

https://github.com/llvm/llvm-project/pull/125965


More information about the llvm-commits mailing list