start newpm migration

[oscardssmith]

No description provided.

[github-actions]

Your PR requires formatting changes to meet the project's style guidelines.
Please consider running Runic (git runic main) to apply these changes.

Click here to view the suggested changes.

diff --git a/src/compiler.jl b/src/compiler.jl
index 82a11400..24e45a4c 100644
--- a/src/compiler.jl
+++ b/src/compiler.jl
@@ -4375,9 +4375,9 @@ function GPUCompiler.compile_unhooked(output::Symbol, job::CompilerJob{<:EnzymeT
         permit_inlining!(f)
     end
 
-    LLVM.@dispose pb=LLVM.NewPMPassBuilder() begin
+    LLVM.@dispose pb = LLVM.NewPMPassBuilder() begin
         @ccall libEnzyme.registerEnzymeAndPassPipeline(pb::Ptr{Cvoid}, 0::UInt8)::Cvoid
-        
+
         LLVM.add!(pb, LLVM.NewPMModulePassManager()) do mpm
             LLVM.add!(mpm, PreserveNVVMPass(true))
         end
diff --git a/src/compiler/optimize.jl b/src/compiler/optimize.jl
index d966d087..046bd4b7 100644
--- a/src/compiler/optimize.jl
+++ b/src/compiler/optimize.jl
@@ -66,7 +66,7 @@ function run_jl_pipeline(pm::ModulePassManager, tm::LLVM.TargetMachine; kwargs..
         end
         return true
     end
-    add!(pm, NewPMModulePass("JLPipeline", jl_pipeline))
+    return add!(pm, NewPMModulePass("JLPipeline", jl_pipeline))
 end
 
 @static if VERSION < v"1.11.0-DEV.428"
@@ -365,7 +365,7 @@ else
             end
             return true
         end
-        add!(pm, NewPMModulePass("CombineMulAdd", combine_mul_add))
+            add!(pm, NewPMModulePass("CombineMulAdd", combine_mul_add))
 end
     end
 end

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 30.23%. Comparing base (df2a49a) to head (2d70da2).
⚠️ Report is 49 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (df2a49a) and HEAD (2d70da2). Click for more details.

HEAD has 14 uploads less than BASE

Flag	BASE (df2a49a)	HEAD (2d70da2)
	17	3

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2475       +/-   ##
===========================================
- Coverage   75.16%   30.23%   -44.94%     
===========================================
  Files          57        2       -55     
  Lines       17821      215    -17606     
===========================================
- Hits        13395       65    -13330     
+ Misses       4426      150     -4276     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[wsmoses]

Let's start to begin with just the preservenvvm being moved over (and leave the others the same, to confirm)

[vchuravy]

replaced by #2675