[all-commits] [llvm/llvm-project] 936c55: [flang][OpenMP] Handle REQUIRES ADMO in lowering (...

[Krzysztof Parzyszek via All-commits]

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

  Changed paths:
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    M flang/lib/Semantics/CMakeLists.txt
    M flang/lib/Semantics/check-omp-structure.cpp
    M flang/lib/Semantics/check-omp-structure.h
    R flang/lib/Semantics/rewrite-directives.cpp
    R flang/lib/Semantics/rewrite-directives.h
    M flang/lib/Semantics/rewrite-parse-tree.cpp
    A flang/test/Lower/OpenMP/requires-admo-acqrel.f90
    A flang/test/Lower/OpenMP/requires-admo-invalid1.f90
    A flang/test/Lower/OpenMP/requires-admo-invalid2.f90
    R flang/test/Semantics/OpenMP/requires-atomic01.f90
    R flang/test/Semantics/OpenMP/requires-atomic02.f90

  Log Message:
  -----------
  [flang][OpenMP] Handle REQUIRES ADMO in lowering (#144362)

The previous approach rewrote the atomic constructs in the AST based on
the REQUIRES ATOMIC_DEFAULT_MEM_ORDER directives. The new approach
checks for incorrect uses of REQUIRED ADMO in the semantic analysis, and
applies it in lowering, eliminating the need for a separate
tree-rewriting procedure.



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