Skip to content

Add sparam#3119

Merged
wsmoses merged 1 commit into
mainfrom
sparam
May 21, 2026
Merged

Add sparam#3119
wsmoses merged 1 commit into
mainfrom
sparam

Conversation

@wsmoses
Copy link
Copy Markdown
Member

@wsmoses wsmoses commented May 21, 2026

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

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/llvm/attributes.jl b/src/llvm/attributes.jl
index eb96fb16..7acc0ff6 100644
--- a/src/llvm/attributes.jl
+++ b/src/llvm/attributes.jl
@@ -1,6 +1,6 @@
 const nofreefns = Set{String}((
-    "jl_f__compute_sparams",
-    "ijl_f__compute_sparams",
+        "jl_f__compute_sparams",
+        "ijl_f__compute_sparams",
     "jl_genericmemory_copyto",
     "jl_get_binding_value_seqcst",
     "ijl_get_binding_value_seqcst",
@@ -217,8 +217,8 @@ const nofreefns = Set{String}((
 ))
 
 const inactivefns = Set{String}((
-    "jl_f__compute_sparams",
-    "ijl_f__compute_sparams",
+        "jl_f__compute_sparams",
+        "ijl_f__compute_sparams",
     
     # The binding lookup is asserted as inactive [e.g. globals are considered constant wrt differentiation]
     # If and when this changes, we need to change that here
diff --git a/test/sparam.jl b/test/sparam.jl
index b129c90c..df366927 100644
--- a/test/sparam.jl
+++ b/test/sparam.jl
@@ -14,4 +14,4 @@ end
         [1.0, 2.0]
     )[1]
     @test res ≈ [1.0, 1.0]
-end
\ No newline at end of file
+end

@github-actions
Copy link
Copy Markdown
Contributor

Benchmark Results

main 0f1ef5a... main / 0f1ef5a...
basics/make_zero/namedtuple 0.0516 ± 0.0025 μs 0.0514 ± 0.0021 μs 1 ± 0.064
basics/make_zero/struct 0.273 ± 0.0054 μs 0.273 ± 0.005 μs 1 ± 0.027
basics/overhead 5.87 ± 0.019 ns 4.33 ± 0.01 ns 1.36 ± 0.0054
basics/remake_zero!/namedtuple 0.223 ± 0.0076 μs 0.224 ± 0.0067 μs 0.999 ± 0.045
basics/remake_zero!/struct 0.226 ± 0.0075 μs 0.226 ± 0.008 μs 0.998 ± 0.049
fold_broadcast/multidim_sum_bcast/1D 10.2 ± 0.49 μs 10.3 ± 1.8 μs 0.994 ± 0.18
fold_broadcast/multidim_sum_bcast/2D 12.1 ± 0.26 μs 12.1 ± 0.27 μs 1 ± 0.031
time_to_load 1.04 ± 0.0084 s 1.06 ± 0.0026 s 0.986 ± 0.0084

Benchmark Plots

A plot of the benchmark results has been uploaded as an artifact at https://github.com/EnzymeAD/Enzyme.jl/actions/runs/26254546213/artifacts/7147939165.

@wsmoses wsmoses merged commit 8461239 into main May 21, 2026
55 of 70 checks passed
@wsmoses wsmoses deleted the sparam branch May 21, 2026 21:56
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