JuliaGPU / Adapt.jl

Other
86 stars 24 forks source link

Optimize LogicalIndices construction. #49

Closed maleadt closed 2 years ago

maleadt commented 2 years ago

Copy the old sum field instead of recalculating it. Should also fix https://github.com/JuliaGPU/CUDA.jl/issues/1304.

codecov[bot] commented 2 years ago

Codecov Report

Merging #49 (e493a03) into master (d905dab) will increase coverage by 1.08%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
+ Coverage   80.39%   81.48%   +1.08%     
==========================================
  Files           5        5              
  Lines          51       54       +3     
==========================================
+ Hits           41       44       +3     
  Misses         10       10              
Impacted Files Coverage Δ
src/wrappers.jl 71.87% <100.00%> (+2.90%) :arrow_up:

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 d905dab...e493a03. Read the comment docs.