[all-commits] [llvm/llvm-project] 1b02cd: [LangRef] Specify that syncscopes can affect the m...

Fabian Ritter via All-commits all-commits at lists.llvm.org
Thu May 28 01:15:40 PDT 2026


  Branch: refs/heads/users/ritter-x2a/03-27-_langref_specify_that_syncscopes_can_affect_the_monotonic_modification_order
  Home:   https://github.com/llvm/llvm-project
  Commit: 1b02cda093819cfc6db4c7546e476f934ac0593e
      https://github.com/llvm/llvm-project/commit/1b02cda093819cfc6db4c7546e476f934ac0593e
  Author: Fabian Ritter <fabian.ritter at amd.com>
  Date:   2026-05-28 (Thu, 28 May 2026)

  Changed paths:
    M llvm/docs/LangRef.rst

  Log Message:
  -----------
  [LangRef] Specify that syncscopes can affect the monotonic modification order

If a target specifies that atomics with mismatching syncscopes appear
non-atomic to each other, there is no point in requiring them to be ordered in
the monotonic modification order. Notably, the [AMDGPU target user
guide](https://llvm.org/docs/AMDGPUUsage.html#memory-scopes) has specified
syncscopes to relax the modification order for years.

So far, I haven't found an example where this less constrained ordering would
be observable (at least with the AMDGPU inclusive scope rules). Whenever a load
would be able to see two monotonic stores with non-inclusive scope, that's
considered a data race (i.e., the load would return `undef`), so it cannot be
used to observe the order of the stores.


  Commit: ae33c4563b54795b75c6a691ce979f2ae99a32cb
      https://github.com/llvm/llvm-project/commit/ae33c4563b54795b75c6a691ce979f2ae99a32cb
  Author: Fabian Ritter <fabian.ritter at amd.com>
  Date:   2026-05-28 (Thu, 28 May 2026)

  Changed paths:
    M llvm/docs/LangRef.rst

  Log Message:
  -----------
  Apply reviewer suggestions

Co-authored-by: Sameer Sahasrabuddhe <sameer.sahasrabuddhe at amd.com>


Compare: https://github.com/llvm/llvm-project/compare/9841f811f322...ae33c4563b54

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list