[all-commits] [llvm/llvm-project] dc0dca: [flang][OpenMP] Allow flush of common block (#139528)

[Tom Eccles via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dc0dcab397ae3de38141e1995e4b4e5e3bb98660
      https://github.com/llvm/llvm-project/commit/dc0dcab397ae3de38141e1995e4b4e5e3bb98660
  Author: Tom Eccles <tom.eccles at arm.com>
  Date:   2025-05-19 (Mon, 19 May 2025)

  Changed paths:
    M flang/lib/Semantics/check-omp-structure.cpp
    A flang/test/Lower/OpenMP/flush-common.f90
    R flang/test/Semantics/OpenMP/flush04.f90

  Log Message:
  -----------
  [flang][OpenMP] Allow flush of common block (#139528)

I think this was denied by accident in
https://github.com/llvm/llvm-project/commit/68180d8d16f07db8200dfce7bae26a80c43ebc5e.

Flush of a common block is allowed by the standard on my reading. It is
not allowed by classic-flang but is supported by gfortran and ifx.

This doesn't need any lowering changes. The LLVM translation ignores the
flush argument list because the openmp runtime library doesn't support
flushing specific data.

Depends upon https://github.com/llvm/llvm-project/pull/139522. Ignore
the first commit in this PR.



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