Skip to content

Conversation

@christiangnrd
Copy link
Member

Backport #2907 to 5.8 branch. Also bumps the patch version

christiangnrd and others added 3 commits October 3, 2025 17:17
We shouldn't perform a dense copy, which we started dispatching to after
the Adapt.jl Union now lists Hermitian as well (Symmetric had been broken before).
@maleadt maleadt changed the title [release-5.8] Fix memory corruption in mapreduce Backports for 5.8 Oct 3, 2025
@maleadt maleadt changed the title Backports for 5.8 Backports for 5.8.5 Oct 3, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2025

Your PR requires formatting changes to meet the project's style guidelines.
Please consider running Runic (git runic release-5.8) to apply these changes.

Click here to view the suggested changes.
diff --git a/lib/cusolver/linalg.jl b/lib/cusolver/linalg.jl
index 2776916f2..ace05099a 100644
--- a/lib/cusolver/linalg.jl
+++ b/lib/cusolver/linalg.jl
@@ -112,9 +112,9 @@ end
 
 # make copyto! for Hermitian and Symmetric dispatch to the Base implementation
 # instead of being overridden by GPUArrays' dense copy (because of AnyGPUArray)
-Base.copyto!(dst::Symmetric{<:Any,<:CuMatrix}, src::Symmetric{<:Any,<:CuMatrix}) =
+Base.copyto!(dst::Symmetric{<:Any, <:CuMatrix}, src::Symmetric{<:Any, <:CuMatrix}) =
     @invoke copyto!(dst::Symmetric, src::Symmetric)
-Base.copyto!(dst::Hermitian{<:Any,<:CuMatrix}, src::Hermitian{<:Any,<:CuMatrix}) =
+Base.copyto!(dst::Hermitian{<:Any, <:CuMatrix}, src::Hermitian{<:Any, <:CuMatrix}) =
     @invoke copyto!(dst::Hermitian, src::Hermitian)
 
 # eigenvalues

@maleadt
Copy link
Member

maleadt commented Oct 3, 2025

I added #2913 to make CI work.

J-C-Q and others added 3 commits October 7, 2025 08:15
@maleadt maleadt merged commit 798eba2 into JuliaGPU:release-5.8 Oct 7, 2025
1 of 2 checks passed
@christiangnrd christiangnrd deleted the backportmapred branch October 7, 2025 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants