Skip to content

Commit a208ef8

Browse files
authored
Preparation for deprecation of __AMDGCN_WAVEFRONT_SIZE (#4311)
See #4270 for more details. Close @4270
1 parent 91527bb commit a208ef8

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

Docs/sphinx_documentation/source/Basics.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,7 @@ reading from command line or input file.
704704

705705
Because many AMReX classes and functions (including destructors
706706
inserted by the compiler) do not function properly after
707-
:cpp:`amrex:Finalize` is called, it's best to put the codes between
707+
:cpp:`amrex::Finalize` is called, it's best to put the codes between
708708
:cpp:`amrex::Initialize` and :cpp:`amrex::Finalize` into its scope
709709
(e.g., a pair of curly braces or a separate function) to make sure
710710
resources are properly freed.

Src/Base/AMReX_GpuDevice.cpp

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,15 @@ namespace {
6565
namespace {
6666
__host__ __device__ void amrex_check_wavefront_size () {
6767
#ifdef __HIP_DEVICE_COMPILE__
68-
// https://github.com/AMReX-Codes/amrex/issues/3792
69-
// __AMDGCN_WAVEFRONT_SIZE is valid in device code only.
70-
// Thus we have to check it this way.
68+
// * https://github.com/AMReX-Codes/amrex/issues/3792
69+
// __AMDGCN_WAVEFRONT_SIZE is valid in device code only.
70+
// Thus we have to check it this way.
71+
// * https://github.com/AMReX-Codes/amrex/issues/4270
72+
// __AMDGCN_WAVEFRONT_SIZE will be deprecated.
73+
#ifdef __AMDGCN_WAVEFRONT_SIZE
7174
static_assert(__AMDGCN_WAVEFRONT_SIZE == AMREX_AMDGCN_WAVEFRONT_SIZE,
7275
"Please let the amrex team know if you encounter this");
76+
#endif
7377
#endif
7478
}
7579
}

0 commit comments

Comments
 (0)