Skip to content

Handle inactive methods#3120

Merged
wsmoses merged 1 commit into
mainfrom
hia
May 23, 2026
Merged

Handle inactive methods#3120
wsmoses merged 1 commit into
mainfrom
hia

Conversation

@wsmoses
Copy link
Copy Markdown
Member

@wsmoses wsmoses commented May 23, 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/internal_rules/inactive.jl b/src/internal_rules/inactive.jl
index fee22851..ccf59e5b 100644
--- a/src/internal_rules/inactive.jl
+++ b/src/internal_rules/inactive.jl
@@ -129,10 +129,10 @@ end
 function EnzymeRules.inactive_noinl(::typeof(Base.dataids), args...)
     return nothing
 end
-function EnzymeRules.inactive_noinl(::typeof(Base.signature_type), args...) 
+function EnzymeRules.inactive_noinl(::typeof(Base.signature_type), args...)
     return nothing
 end
-function EnzymeRules.inactive_noinl(::typeof(Base.methods), args...) 
+function EnzymeRules.inactive_noinl(::typeof(Base.methods), args...)
     return nothing
 end
 

@github-actions
Copy link
Copy Markdown
Contributor

Benchmark Results

main 55227a3... main / 55227a3...
basics/make_zero/namedtuple 0.0516 ± 0.0018 μs 0.051 ± 0.0016 μs 1.01 ± 0.048
basics/make_zero/struct 0.257 ± 0.0055 μs 0.258 ± 0.0058 μs 0.997 ± 0.031
basics/overhead 4.95 ± 0.001 ns 4.03 ± 0.01 ns 1.23 ± 0.0031
basics/remake_zero!/namedtuple 0.224 ± 0.0063 μs 0.225 ± 0.008 μs 0.994 ± 0.045
basics/remake_zero!/struct 0.221 ± 0.006 μs 0.222 ± 0.0077 μs 0.992 ± 0.044
fold_broadcast/multidim_sum_bcast/1D 10.2 ± 0.79 μs 10.3 ± 0.96 μs 0.995 ± 0.12
fold_broadcast/multidim_sum_bcast/2D 12.1 ± 0.24 μs 12.1 ± 0.25 μs 0.997 ± 0.028
time_to_load 1.03 ± 0.0058 s 1.04 ± 0.0036 s 0.995 ± 0.0066

Benchmark Plots

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

@wsmoses wsmoses merged commit 000acf6 into main May 23, 2026
53 of 70 checks passed
@wsmoses wsmoses deleted the hia branch May 23, 2026 15:52
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