Try to improve test coverage slightly.

Author: chriselrod

src/add_compute.jl

@@ -90,16 +90,16 @@ end
     # @assert iscompute(op)
     # pushop!(ls, child, name(op))
 # end
-function isreductzero(op::Operation, ls::LoopSet, reduct_zero::Symbol)
-    isconstant(op) || return false
-    reduct_zero === op.instruction.mod && return true
-    if reduct_zero === :zero
-        iszero(ls, op) && return true
-    elseif reduct_zero === :one
-        isone(ls, op) && return true
-    end
-    false
-end
+# function isreductzero(op::Operation, ls::LoopSet, reduct_zero::Symbol)
+#     isconstant(op) || return false
+#     reduct_zero === op.instruction.mod && return true
+#     if reduct_zero === :zero
+#         iszero(ls, op) && return true
+#     elseif reduct_zero === :one
+#         isone(ls, op) && return true
+#     end
+#     false
+# end
 
 # function substitute_op_in_parents!(vparents::Vector{Operation}, replacer::Operation, replacee::Operation)
 #     for i ∈ eachindex(vparents)

src/graphs.jl

@@ -11,10 +11,10 @@ function UnrollSpecification(us::UnrollSpecification, u₁, u₂)
     @unpack u₁loopnum, u₂loopnum, vectorizedloopnum = us
     UnrollSpecification(u₁loopnum, u₂loopnum, vectorizedloopnum, u₁, u₂)
 end
-function UnrollSpecification(us::UnrollSpecification; u₁ = us.u₁, u₂ = us.u₂)
-    @unpack u₁loopnum, u₂loopnum, vectorizedloopnum = us
-    UnrollSpecification(u₁loopnum, u₂loopnum, vectorizedloopnum, u₁, u₂)
-end
+# function UnrollSpecification(us::UnrollSpecification; u₁ = us.u₁, u₂ = us.u₂)
+#     @unpack u₁loopnum, u₂loopnum, vectorizedloopnum = us
+#     UnrollSpecification(u₁loopnum, u₂loopnum, vectorizedloopnum, u₁, u₂)
+# end
 isunrolled1(us::UnrollSpecification, n::Int) = us.u₁loopnum == n
 isunrolled2(us::UnrollSpecification, n::Int) = !isunrolled1(us, n) && us.u₂loopnum == n
 isvectorized(us::UnrollSpecification, n::Int) = us.vectorizedloopnum == n
@@ -225,20 +225,20 @@ function zerotype(ls::LoopSet, op::Operation)
     end
     INVALID
 end
-function Base.iszero(ls::LoopSet, op::Operation)
-    opid = identifier(op)
-    for (id,_) ∈ ls.preamble_zeros
-        opid == id && return true
-    end
-    false
-end
-function Base.isone(ls::LoopSet, op::Operation)
-    opid = identifier(op)
-    for (id,_) ∈ ls.preamble_ones
-        opid == id && return true
-    end
-    false
-end
+# function Base.iszero(ls::LoopSet, op::Operation)
+#     opid = identifier(op)
+#     for (id,_) ∈ ls.preamble_zeros
+#         opid == id && return true
+#     end
+#     false
+# end
+# function Base.isone(ls::LoopSet, op::Operation)
+#     opid = identifier(op)
+#     for (id,_) ∈ ls.preamble_ones
+#         opid == id && return true
+#     end
+#     false
+# end
 
 
 includesarray(ls::LoopSet, array::Symbol) = array ∈ ls.includedarrays

test/special.jl

@@ -12,7 +12,7 @@
     end
     function myvexpavx!(b, a)
         @avx for i ∈ eachindex(a)
-            b[i] = exp(a[i])
+            b[i] = Base.exp(a[i])
         end
     end
     function myvexp_avx!(b, a)