Closed Kunde21 closed 7 years ago
This won't merge. Would you fix that please.
@kortschak I tried to trigger the errors, but didn't get any. We can revert if you had other things to address.
This breaks "gonum/blas/native". I'm going to revert it. Please don't merge until there is an approval.
Testing code was cleaned up and updated to use sigil values in the guards instead of NaN. Helper functions were consolidated. Started to transition benchmarks to the cleaner go1.7 sub-benchmark construct.
The larger loop does slow down the small cases,
len(vec) <= 10
, compared to the old asm while speeding up the everything up to the memory bottleneck.Benchmarks against old code
Benchmarks vs for loop