[llvm] [Pass] Add `eraseIf` in pass managers and adaptors (PR #116912)

Akshat Oke via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 13:19:32 PST 2024


optimisan wrote:

> the [legacy pass manager does the filtering when adding passes](https://github.com/llvm/llvm-project/blob/905e831f8c8341e53e7e3adc57fd20b8e08eb999/llvm/lib/CodeGen/TargetPassConfig.cpp#L712) rather than after the fact, and I like that better than examining passes after the fact

Any particular reason for that? IIUC adding passes in the legacy PM is costlier than in NPM
But I don't get if there's a specific motivation in NPM to keep the same behaviour to filter only while adding passes.

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


More information about the llvm-commits mailing list