[llvm] [NewPM] Remove LowerWidenableConditionLegacyPass (PR #72818)

[Aiden Grossman via llvm-commits]

boomanaiden154 wrote:

> Azul confirmed in https://reviews.llvm.org/D153677#4452433 that they're not using this in the backend, so I think we can go ahead and remove this. If it turns out that someone else is using it in the backend, we should revert the patch later.

Sounds good. One of the main reasons I kept everything split up was to make reverts easy for cases like that.

> I think at least the printing passes should get migrated to `print<foo>` NewPM passes.

Can you clarify what you mean here? Are you referring to adjusting `PassRegistry.def` to switch the `print-*` names to `print<*>` names?

And then the plan right now would be to keep the legacy versions of all these utility passes? Or should we look at removing them too?

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