Skip to content

Commit b32b642

Browse files
authored
SWDEV-530921 - Remove the usage of __AMDGCN_WAVEFRONT_SIZE as compile time constant (#82)
1 parent e3cb539 commit b32b642

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

docs/how-to/hip_cpp_language_extensions.rst

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,8 @@ compile-time constant on the host. It has to be queried using
467467

468468
``warpSize`` should not be assumed to be a specific value in portable HIP
469469
applications. NVIDIA devices return 32 for this variable; AMD devices return
470-
64 for gfx9 and 32 for gfx10 and above. While code that assumes a ``warpSize``
470+
64 for gfx9 and 32 for gfx10 and above. HIP doesn't support ``warpSize`` of
471+
64 on gfx10 and above. While code that assumes a ``warpSize``
471472
of 32 can run on devices with a ``warpSize`` of 64, it only utilizes half of
472473
the the compute resources.
473474

include/hip/hip_runtime.h

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
Copyright (c) 2015 - 2021 Advanced Micro Devices, Inc. All rights reserved.
2+
Copyright (c) 2015 - 2025 Advanced Micro Devices, Inc. All rights reserved.
33
44
Permission is hereby granted, free of charge, to any person obtaining a copy
55
of this software and associated documentation files (the "Software"), to deal
@@ -37,10 +37,6 @@ THE SOFTWARE.
3737
#ifndef HIP_INCLUDE_HIP_HIP_RUNTIME_H
3838
#define HIP_INCLUDE_HIP_HIP_RUNTIME_H
3939

40-
#if __HIP_DEVICE_COMPILE__ && !__GFX7__ && !__GFX8__ && !__GFX9__ && __AMDGCN_WAVEFRONT_SIZE == 64
41-
#error HIP is not supported on the specified GPU ARCH with wavefront size 64
42-
#endif
43-
4440
#if !defined(__HIPCC_RTC__)
4541
// Some standard header files, these are included by hc.hpp and so want to make them avail on both
4642
// paths to provide a consistent include env and avoid "missing symbol" errors that only appears

0 commit comments

Comments
 (0)