Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

copyto! fix for BitArray/AbstractArray #46161

Merged
merged 3 commits into from
Jul 25, 2022
Merged

Conversation

N5N3
Copy link
Member

@N5N3 N5N3 commented Jul 25, 2022

This PR fix #25968 via dispatching copyto! to Base.unsafe_copyto!
Also the missing aliasing check is added to the general fallback for AbstractArray.
Test added.

@N5N3 N5N3 added domain:arrays [a, r, r, a, y, s] kind:bugfix This change fixes an existing bug backport 1.6 Change should be backported to release-1.6 backport 1.8 Change should be backported to release-1.8 labels Jul 25, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`
base/abstractarray.jl Outdated Show resolved Hide resolved
@fredrikekre fredrikekre merged commit 0ea2b2d into JuliaLang:master Jul 25, 2022
@N5N3 N5N3 deleted the copyto! branch July 26, 2022 01:06
KristofferC pushed a commit that referenced this pull request Aug 6, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
@KristofferC KristofferC removed the backport 1.8 Change should be backported to release-1.8 label Aug 7, 2022
ffucci pushed a commit to ffucci/julia that referenced this pull request Aug 11, 2022
…uliaLang#46161)

1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`
pcjentsch pushed a commit to pcjentsch/julia that referenced this pull request Aug 18, 2022
…uliaLang#46161)

1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
KristofferC pushed a commit that referenced this pull request Oct 10, 2023
1. map `copyto!(::BitArray, n1, ::BitArray, n2, l)` to `Base.unsafe_copyto!`
2. add missing unaliasing in `copyto!` for `AbstractArray`

(cherry picked from commit 0ea2b2d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.6 Change should be backported to release-1.6 domain:arrays [a, r, r, a, y, s] kind:bugfix This change fixes an existing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

copyto! dispatches incorrectly for BitVector
3 participants