[PATCH] D128830: [Pipelines] Introduce DAE after ArgumentPromotion

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

There should be a PhaseOrdering test that shows that the argument is not removed in the current opimization pipeline.



================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:775
+  if (Level == OptimizationLevel::O3)
+    MIWP.addLateModulePass(DeadArgumentEliminationPass());
+
----------------
I don't think we need a late module pass for this, it can be part of the normal module optimization pipeline (post module simplification).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128830