Remove print statement, and fix bug in no-dep-chain breakup.

JuliaSIMD · Jan 17, 2020 · 4f061f8 · 4f061f8 · chriselrod · Jan 17, 2020
1 parent 5b09fe9
commit 4f061f8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/determinestrategy.jl b/src/determinestrategy.jl
@@ -131,10 +131,10 @@ function unroll_no_reductions(ls, order, vectorized, Wshift, size_T)
         end
     end
     # heuristic guess
-    @show compute_rt, load_rt
+    # @show compute_rt, load_rt
     # roundpow2(min(4, round(Int, (compute_rt + load_rt + 1) / compute_rt)))
     rt = max(compute_rt, load_rt)
-    roundpow2( min( 4, round(Int, 16 / rt) ) )
+    max(1, roundpow2( min( 4, round(Int, 16 / rt) ) ))
 end
 function determine_unroll_factor(
     ls::LoopSet, order::Vector{Symbol}, unrolled::Symbol, vectorized::Symbol = first(order)

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -1008,14 +1008,13 @@ end
         a = rand(T, M); B = rand(T, M, N); c = rand(T, N); c′ = c';
         d1 =      @. a + B * c′;
         d2 = @avx @. a + B * c′;
-
         @test d1 ≈ d2
+
         @.      d1 = a + B * c′;
         @avx @. d2 = a + B * c′;
         @test d1 ≈ d2
 
         d3 = a .+ B * c;
-        # no method matching _similar_for(::UnitRange{Int64}, ::Type{Any}, ::Product)
         d4 = @avx a .+ B *ˡ c;
         @test d3 ≈ d4