Skip to content

Commit 1d29f3e

Browse files
committed
[AMDGPU] Drop docs for invalid load-release and store-acquire operations
The LangRef says "release and acq_rel orderings are not valid on load instructions" [for loads](https://llvm.org/docs/LangRef.html#load-instruction) and "acquire and acq_rel orderings aren't valid on store instructions" [for stores](https://llvm.org/docs/LangRef.html#store-instruction). Providing them in textual IR is diagnosed with an error. Therefore, we should not define semantics for these invalid constructs. Part of LCOMPILER-2273.
1 parent d11a719 commit 1d29f3e

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

llvm/docs/AMDGPUUsage.rst

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7214,15 +7214,6 @@ orderings (``acquire``, ``release``, ``acq_rel``, or ``seq_cst``).
72147214
The memory model does not support the region address space which is treated as
72157215
non-atomic.
72167216

7217-
Acquire memory ordering is not meaningful on store atomic instructions and is
7218-
treated as non-atomic.
7219-
7220-
Release memory ordering is not meaningful on load atomic instructions and is
7221-
treated as non-atomic.
7222-
7223-
Acquire-release memory ordering is not meaningful on load or store atomic
7224-
instructions and is treated as acquire and release respectively.
7225-
72267217
The memory order also adds the single thread optimization constraints defined in
72277218
table
72287219
:ref:`amdgpu-amdhsa-memory-model-single-thread-optimization-constraints-table`.

0 commit comments

Comments
 (0)