Skip to content

Commit b07dba7

Browse files
refactor: avoid unnecessary dynamic-ness in cse_inside_expr
1 parent c9a8171 commit b07dba7

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/code.jl

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -869,10 +869,9 @@ end
869869
"""
870870
$(TYPEDSIGNATURES)
871871
872-
Return `true` if CSE should descend inside `sym`, which has operation `f` and
873-
arguments `args...`.
872+
Return `true` if CSE should descend inside `sym`, which has operation `f`.
874873
"""
875-
function cse_inside_expr(sym, f, args...)
874+
function cse_inside_expr(sym, f)
876875
return true
877876
end
878877

@@ -1009,7 +1008,7 @@ function cse!(expr::BasicSymbolic{T}, state::CSEState) where {T}
10091008
if op isa BasicSymbolic{T}
10101009
SymbolicUtils.is_function_symbolic(op) || return expr
10111010
end
1012-
cse_inside_expr(expr, op, args...) || return expr
1011+
cse_inside_expr(expr, op) || return expr
10131012
args = map(Base.Fix2(cse!, state), args)
10141013
# use `term` instead of `maketerm` because we only care about the operation being performed
10151014
# and not the representation. This avoids issues with `newsym` symbols not having sizes, etc.

0 commit comments

Comments
 (0)