Skip to content

Commit 9637b0e

Browse files
committed
lowering: Don't implicitly @nospecialize macros
There's not much purpose of this de-specialization, since macros are generally excluded from inference / compilation anyway (except for JuliaLang#57833 which was a bug) Since our AST is already relatively homogeneous, it's not clear that we gained much by this de-specialization when it was used. Arbitrarily de-specializing a function also often leads to bad (wide) edges in inference etc., causing frequent invalidations.
1 parent 3360a44 commit 9637b0e

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

src/julia-syntax.scm

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1347,11 +1347,7 @@
13471347
`(function (call ,(macroify-name (cadr (cadr e)))
13481348
(|::| __source__ (core LineNumberNode))
13491349
(|::| __module__ (core Module))
1350-
,@(map (lambda (v)
1351-
(if (symbol? v)
1352-
`(meta nospecialize ,v)
1353-
v))
1354-
anames))
1350+
,@anames)
13551351
,@(cddr e)))))
13561352
((and (length= e 2) (valid-macro-def-name? (cadr e)))
13571353
(expand-forms `(function ,(macroify-name (cadr e)))))

0 commit comments

Comments
 (0)