Skip to content

Avoid blow up of function pre-amble on higer order AD#2350

Merged
vchuravy merged 1 commit intomainfrom
vc/nosafepoint
Apr 9, 2025
Merged

Avoid blow up of function pre-amble on higer order AD#2350
vchuravy merged 1 commit intomainfrom
vc/nosafepoint

Conversation

@vchuravy
Copy link
Member

@vchuravy vchuravy commented Apr 8, 2025

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 8, 2025

Benchmark Results

main 6169d9c... main/6169d9c95f35ef...
basics/overhead 4.64 ± 0.011 ns 4.33 ± 0.01 ns 1.07
time_to_load 1.11 ± 0.0074 s 1.1 ± 0.008 s 1.01

Benchmark Plots

A plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR.
Go to "Actions"->"Benchmark a pull request"->[the most recent run]->"Artifacts" (at the bottom).

@vchuravy vchuravy marked this pull request as ready for review April 9, 2025 14:50
@vchuravy vchuravy merged commit 6059bb3 into main Apr 9, 2025
23 of 32 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Apr 9, 2025

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 b07be0c..67c566e 100644
--- a/src/compiler.jl
+++ b/src/compiler.jl
@@ -1328,7 +1328,7 @@ struct PrimalCompilerParams <: AbstractEnzymeCompilerParams
 end
 
 # Avoid blow-up of higer-order AD
-GPUCompiler.can_safepoint(::CompilerJob{<:Any,<:AbstractEnzymeCompilerParams}) = false
+GPUCompiler.can_safepoint(::CompilerJob{<:Any, <:AbstractEnzymeCompilerParams}) = false
 
 DefaultCompilerTarget(; kwargs...) =
     GPUCompiler.NativeCompilerTarget(; jlruntime = true, kwargs...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant