[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