-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Type instability with broadcast #1078
Comments
At least part of the issue with complicated broadcasts (that example is complicated enough) and StaticArrays is that Julia gives up way too easily when faced with method instance recursion inlining. My attempts at reducing the amount of method instance recursion in broadcast code weren't met with much enthusiasm, and I haven't seen any improvements on the compiler side (very recent master branch of Julia has the same issue). |
JuliaLang/julia#43322 should be enough to fix all related problems. Do we want to port that design into |
So I'm not the only one who is trying to solve that issue 🙂 . Your changes look fine at the first glance so I'd review them for StaticArrays.jl and be OK with merging them, though it would have to be limited to |
Stable:
Type-unstable:
Related: SciML/Integrals.jl#94
CC: @YingboMa @ChrisRackauckas
The text was updated successfully, but these errors were encountered: