search

JuliaLinearAlgebra / Octavian.jl

Multi-threaded BLAS-like library that provides pure Julia matrix multiplication
https://julialinearalgebra.github.io/Octavian.jl/stable/
Other
226 stars 18 forks source link

fix a potential bounds error #148

Closed andrewjradcliffe closed 2 years ago

andrewjradcliffe commented 2 years ago

Truly, very minor. I happened upon this when searching for @turbo uses.

chriselrod commented 2 years ago

Thanks, definitely shouldn't be using 1:length(C).

codecov[bot] commented 2 years ago

Codecov Report

Merging #148 (7817fd0) into master (ad7f099) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #148   +/-   ##
=======================================
  Coverage   88.06%   88.06%           
=======================================
  Files          13       13           
  Lines         930      930           
=======================================
  Hits          819      819           
  Misses        111      111
Impacted Files Coverage Δ
src/matmul.jl 91.84% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ad7f099...7817fd0. Read the comment docs.