Broadcast shouldn't use inbounds when calling f

Previously, `broadcast(getindex, A, I...)` would call `getindex` within an
at-inbounds context. This patch simply moves the function call so that it
will throw a BoundsError appropriately.

base/broadcast.jl

@@ -140,7 +140,8 @@ end
             # extract array values
             @nexprs $nargs i->(@inbounds val_i = _broadcast_getindex(A_i, I_i))
             # call the function and store the result
-            @inbounds B[I] = @ncall $nargs f val
+            result = @ncall $nargs f val
+            @inbounds B[I] = result
         end
     end
 end

test/boundscheck_exec.jl

@@ -174,4 +174,9 @@ if bc_opt != bc_off
     @test_throws BoundsError k1(Int[])
 end
 
+# Ensure that broadcast doesn't use @inbounds when calling the function
+let A = zeros(3,3)
+    @test_throws BoundsError broadcast(getindex, A, 1:3, 1:3)
+end
+
 end