Skip to content

Commit

Permalink
Update comments about RAJA_MAX_ALIGN
Browse files Browse the repository at this point in the history
  • Loading branch information
CRobeck authored Dec 13, 2022
1 parent 4fdf583 commit 5d458ce
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions include/RAJA/config.hpp.in
Original file line number Diff line number Diff line change
Expand Up @@ -242,11 +242,11 @@ static_assert(RAJA_HAS_SOME_CXX14,


/* NOTE: Below we define RAJA_MAX_ALIGN for each compiler, currently it is set as 16 bytes
for all cases. Previously this was set by alignof(std::max_align_t) which, in Clang,
for all cases, except MSVC. Previously this was set by alignof(std::max_align_t) which, in Clang,
is based on the sizeof(long double). This causes an in inconsistency as CUDA/HIP long doubles
are demoted to doubles causing alignof(std::max_align_t) to return 8 bytes on the device and
16 bytes on the host. We therefore set a standard size and ensure validity through a
static_assert in pattern/WorkGroup/WorkStruct.hpp.
static_assert.
*/

namespace RAJA {
Expand Down

0 comments on commit 5d458ce

Please sign in to comment.