[all-commits] [llvm/llvm-project] eba63c: [flang][OpenMP] Improve handling of REQUIRES ATOMI...

[Krzysztof Parzyszek via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: eba63cd76f7ba7f9e9964b1263f76409d08fcd04
      https://github.com/llvm/llvm-project/commit/eba63cd76f7ba7f9e9964b1263f76409d08fcd04
  Author: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
  Date:   2025-06-13 (Fri, 13 Jun 2025)

  Changed paths:
    M flang/lib/Semantics/rewrite-directives.cpp
    A flang/test/Lower/OpenMP/requires-atomic-default-mem-order.f90
    M flang/test/Semantics/OpenMP/requires-atomic02.f90

  Log Message:
  -----------
  [flang][OpenMP] Improve handling of REQUIRES ATOMIC_DEFAULT_MEM_ORDER (#143917)

According to OpenMP 5.0 rules, the ACQ_REL ordering coming from a
REQUIRES directive may need to be replaced with ACQUIRE or RELEASE
depending on the directive in the ATOMIC construct. This was not done,
leading to an incorrect "memory-order" clause appearing in the generated
HLFIR.

This may need to be relaxed a bit to fully comply with later spec
versions, that will be done in a future PR.



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