Fix performance regression in broadcasting with CartesianIndices (Jul…

…iaLang#39333) * Fix performance regression in broadcasting with CartesianIndices This avoids the boundary check due to a change in the implementation of iteration using `CartecianIndices` in PR JuliaLang#37829. This is a workaround on the caller side and does not change the iteration mechanism itself. Co-authored-by: Matt Bauman <[email protected]> Co-authored-by: thofma <[email protected]>
antoine-levitt · May 9, 2021 · 3e6e276 · 3e6e276
1 parent cf42328
commit 3e6e276
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/base/broadcast.jl b/base/broadcast.jl
@@ -977,8 +977,10 @@ preprocess_args(dest, args::Tuple{}) = ()
         end
     end
     bc′ = preprocess(dest, bc)
-    @simd for I in eachindex(bc′)
-        @inbounds dest[I] = bc′[I]
+    # Performance may vary depending on whether `@inbounds` is placed outside the
+    # for loop or not. (cf. https://github.com/JuliaLang/julia/issues/38086)
+    @inbounds @simd for I in eachindex(bc′)
+        dest[I] = bc′[I]
     end
     return dest
 end