Don't use estrin for VecUnrolls

JuliaSIMD · Jan 10, 2021 · 41e600b · 41e600b · chriselrod · Jan 10, 2021
1 parent 2dd30f4
commit 41e600b
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/src/estrin.jl b/src/estrin.jl
@@ -44,15 +44,17 @@
     end
     ex
 end
+# Given a `VecUnroll` argument, we'll instruction level parallelism that way and can thus forgo `estrin` 
+@inline estrin(x::VecUnroll, p::NTuple{N}) where {N} = evalpoly(x, p)
 
-macro estrin(x, p...)
-    t = Expr(:tuple); foreach(pᵢ -> push!(t.args, pᵢ), p)
-    if __module__ == SLEEFPirates
-        esc(Expr(:call, :estrin, x, t))
-    else
-        esc(Expr(:call, Expr(:(.), :SLEEFPirates, QuoteNode(:estrin)), x, t))
-    end
-end
+# macro estrin(x, p...)
+#     t = Expr(:tuple); foreach(pᵢ -> push!(t.args, pᵢ), p)
+#     if __module__ == SLEEFPirates
+#         esc(Expr(:call, :estrin, x, t))
+#     else
+#         esc(Expr(:call, Expr(:(.), :SLEEFPirates, QuoteNode(:estrin)), x, t))
+#     end
+# end
 
 macro horner(x, p...)
     N = length(p)