Skip to content

Commit 53b4830

Browse files
authored
[AMDGPU] Tweak address space definitions. NFC. (#133442)
Define address spaces in numerical order. Fix comments to refer to "local" instead of "group" address space.
1 parent d6dcd98 commit 53b4830

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

llvm/include/llvm/Support/AMDGPUAddrSpace.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ enum : unsigned {
3131
GLOBAL_ADDRESS = 1, ///< Address space for global memory (RAT0, VTX0).
3232
REGION_ADDRESS = 2, ///< Address space for region memory. (GDS)
3333

34-
CONSTANT_ADDRESS = 4, ///< Address space for constant memory (VTX2).
3534
LOCAL_ADDRESS = 3, ///< Address space for local memory.
35+
CONSTANT_ADDRESS = 4, ///< Address space for constant memory (VTX2).
3636
PRIVATE_ADDRESS = 5, ///< Address space for private memory.
3737

3838
CONSTANT_ADDRESS_32BIT = 6, ///< Address space for 32-bit constant memory.

llvm/lib/Target/AMDGPU/AMDGPU.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -537,11 +537,11 @@ static inline bool addrspacesMayAlias(unsigned AS1, unsigned AS2) {
537537
// This array is indexed by address space value enum elements 0 ... to 9
538538
// clang-format off
539539
static const bool ASAliasRules[10][10] = {
540-
/* Flat Global Region Group Constant Private Const32 BufFatPtr BufRsrc BufStrdPtr */
540+
/* Flat Global Region Local Constant Private Const32 BufFatPtr BufRsrc BufStrdPtr */
541541
/* Flat */ {true, true, false, true, true, true, true, true, true, true},
542542
/* Global */ {true, true, false, false, true, false, true, true, true, true},
543543
/* Region */ {false, false, true, false, false, false, false, false, false, false},
544-
/* Group */ {true, false, false, true, false, false, false, false, false, false},
544+
/* Local */ {true, false, false, true, false, false, false, false, false, false},
545545
/* Constant */ {true, true, false, false, false, false, true, true, true, true},
546546
/* Private */ {true, false, false, false, false, true, false, false, false, false},
547547
/* Constant 32-bit */ {true, true, false, false, true, false, false, true, true, true},

0 commit comments

Comments
 (0)