Skip to content

Commit 8b8f087

Browse files
committed
Unroll by 4 as a dumb heuristic more often.
1 parent 97d5721 commit 8b8f087

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/modeling/determinestrategy.jl

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -264,13 +264,11 @@ function unroll_no_reductions(ls, order, vloopsym)
264264
# heuristic guess
265265
# roundpow2(min(4, round(Int, (compute_rt + load_rt + 1) / compute_rt)))
266266
memory_rt = load_rt + store_rt
267-
# @show memory_rt, load_rt, store_rt, compute_rt, compute_l
268-
269-
u = if compute_rt > memory_rt
267+
u = if compute_rt 1
268+
4
269+
elseif compute_rt > memory_rt
270270
clamp(round(Int, compute_l / compute_rt), 1, 4)
271271
# max(1, VectorizationBase.nextpow2( min( 4, round(Int, 8 / compute_rt) ) ))
272-
elseif iszero(compute_rt)
273-
4
274272
elseif iszero(load_rt)
275273
iszero(store_rt) ? 4 : max(1, min(4, round(Int, 2compute_rt / store_rt)))
276274
else

0 commit comments

Comments
 (0)