[all-commits] [llvm/llvm-project] 0de2cc: [flang][OpenMP]Improve support for DECLARE REDUCTI...

[Mats Petersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0de2ccab7be85aeeefcd5757d29126246c373731
      https://github.com/llvm/llvm-project/commit/0de2ccab7be85aeeefcd5757d29126246c373731
  Author: Mats Petersson <mats.petersson at arm.com>
  Date:   2025-02-19 (Wed, 19 Feb 2025)

  Changed paths:
    M flang/include/flang/Parser/parse-tree.h
    M flang/lib/Parser/openmp-parsers.cpp
    M flang/lib/Parser/unparse.cpp
    M flang/lib/Semantics/check-omp-structure.cpp
    M flang/lib/Semantics/resolve-directives.cpp
    M flang/lib/Semantics/resolve-names.cpp
    M flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90
    A flang/test/Parser/OpenMP/declare-reduction-unparse.f90
    M flang/test/Semantics/OpenMP/declarative-directive01.f90
    A flang/test/Semantics/OpenMP/declare-reduction.f90

  Log Message:
  -----------
  [flang][OpenMP]Improve support for DECLARE REDUCTION (#127088)

Part of the DECLARE REDUCTION was already supported by the parser, but
the semantics to add the reduction identifier wasn't implemented.

The semantics would not accept the name given by the reduction, so a few
lines added to support that.

Some tests were in place but not quite working, so fixed those up too.
Adding new tests for unparsing and parse-tree, as well as checking the
symbolic name being generated.

Lowering of DECLARE REDUCTION is not supported in this patch, and a test
that it hits the relevant TODO is in this patch (most of this was
already existing, but not actually testing the TODO message).



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