[llvm] Reland "[LTO] Run Argument Promotion before IPSCCP" (PR #111853)
Hari Limaye via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 21 06:48:57 PDT 2024
hazzlim wrote:
> In any case, this change needs a PhaseOrdering test.
Thanks for pointing this out - I've added a PhaseOrdering test.
> I see that there is a PostOrderFunctionAttrsPass run shortly after these passes -- would it be feasible to move it earlier rather than running twice?
Good point - I've moved the `PostOrderFunctionAttrsPass` from after IPSCCP to before, and this doesn't seem to break any PhaseOrdering tests or cause any regressions in SPEC2017.
https://github.com/llvm/llvm-project/pull/111853
More information about the llvm-commits
mailing list