[llvm] [SPIR-V][NewPM] Fix SPIRVEmitIntrinsics registration with the new pass manager (PR #209966)

Arseniy Obolenskiy via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 16 02:17:48 PDT 2026


aobolensk wrote:

> Is there a way to test the npm path? Otherwise it looks good to me.

The test is already there (use of `opt -passes="<pass-name>"` in test line is enough)

There is a misleading PR name: NewPM initial registration was done in https://github.com/llvm/llvm-project/pull/188285, but this wrapper was kinda fake, it instantiated the full ModulePass (`class SPIRVEmitIntrinsics : public ModulePass, InstVisitor<...>`) just to call runOnModule() directly (NewPM logic), skipping PassManager entirely

I have corrected the PR title

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


More information about the llvm-commits mailing list