[PATCH] D88293: [ControlDependentDCE] Add Control Dependent DCE pass

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I guess I missed the initial conversation on this, apologies for that.
Can you explain in more detail why this is as standalone pass.
DCE, in general, is something I would like us to do more aggressively in the IPO setting so I'm a little worried to introduce more function passes.
Also, IPSCCP seems to perform well on one test cases already https://godbolt.org/z/sM9zr8 , maybe the second one is not far out of reach (@fhahn?)
We also have other places where we do this kind of reasoning and that could do the job, hopefully in a more holistic manner.


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

https://reviews.llvm.org/D88293