Skip to content

adds <string.h> includes for ROCm 7 compatibility#1857

Merged
danielpeter merged 1 commit intoSPECFEM:develfrom
AMD-HPC:rocm7support
Feb 13, 2026
Merged

adds <string.h> includes for ROCm 7 compatibility#1857
danielpeter merged 1 commit intoSPECFEM:develfrom
AMD-HPC:rocm7support

Conversation

@frawanye
Copy link

Without these includes, compiling with rocm 7 causes the following error: no matching function for call to 'memcpy'
in the following files:

  • src/gpu/assemble_MPI_vector_cuda.cu
  • src/gpu/lts_assembly_mpi_cuda.cu
  • src/gpu/prepare_mesh_constants_cuda.cu

Without these includes, compiling with rocm 7 causes the following error:
no matching function for call to 'memcpy'
in the following files:
- src/gpu/assemble_MPI_vector_cuda.cu
- src/gpu/lts_assembly_mpi_cuda.cu
- src/gpu/prepare_mesh_constants_cuda.cu
@danielpeter
Copy link
Member

thanks for the update!

as this didn't show up so far, what system were you using to compile the code?

anyway, i'll merge it and then probably move these include statements #include <string.h> to the file mesh_constants_gpu.h for convenience and add some HIP compilation checks.

thanks again,
-daniel

@danielpeter danielpeter merged commit ece6276 into SPECFEM:devel Feb 13, 2026
41 checks passed
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.

2 participants