[all-commits] [llvm/llvm-project] e184c0: [flang][OpenMP] Use new modifier infrastructure fo...

[Krzysztof Parzyszek via All-commits]

  Branch: refs/heads/users/kparzysz/spr/m06-map-motion
  Home:   https://github.com/llvm/llvm-project
  Commit: e184c0b477bc22e9211bdd5985d227a6848e6456
      https://github.com/llvm/llvm-project/commit/e184c0b477bc22e9211bdd5985d227a6848e6456
  Author: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
  Date:   2024-11-23 (Sat, 23 Nov 2024)

  Changed paths:
    M flang/examples/FeatureList/FeatureList.cpp
    M flang/examples/FlangOmpReport/FlangOmpReportVisitor.cpp
    M flang/examples/FlangOmpReport/FlangOmpReportVisitor.h
    M flang/include/flang/Parser/dump-parse-tree.h
    M flang/include/flang/Parser/parse-tree.h
    M flang/include/flang/Semantics/openmp-modifiers.h
    M flang/lib/Lower/OpenMP/ClauseProcessor.cpp
    M flang/lib/Lower/OpenMP/Clauses.cpp
    M flang/lib/Lower/OpenMP/Clauses.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/check-omp-structure.h
    M flang/lib/Semantics/openmp-modifiers.cpp
    M flang/lib/Semantics/resolve-directives.cpp
    M flang/lib/Semantics/resolve-names.cpp
    M flang/test/Lower/OpenMP/Todo/map-mapper.f90
    M flang/test/Parser/OpenMP/from-clause.f90
    M flang/test/Parser/OpenMP/map-modifiers.f90
    M flang/test/Parser/OpenMP/target-update-to-clause.f90
    M flang/test/Semantics/OpenMP/combined-constructs.f90
    M flang/test/Semantics/OpenMP/defaultmap-clause-v45.f90
    M flang/test/Semantics/OpenMP/from-clause-v45.f90
    M flang/test/Semantics/OpenMP/from-clause-v51.f90
    M flang/test/Semantics/OpenMP/map-clause.f90
    M flang/test/Semantics/OpenMP/map-modifiers.f90
    M flang/test/Semantics/OpenMP/to-clause-v45.f90
    M flang/test/Semantics/OpenMP/to-clause-v51.f90

  Log Message:
  -----------
  [flang][OpenMP] Use new modifier infrastructure for MAP/FROM/TO clauses

This removes the specialized parsers and helper classes for these clauses,
namely ConcatSeparated, MapModifiers, and MotionModifiers. Map and the
motion clauses are now handled in the same way as all other clauses with
modifiers, with one exception: the commas separating their modifiers are
optional. This syntax is deprecated in OpenMP 5.2.

Implement version checks for modifiers: for a given modifier on a given
clause, check if that modifier is allowed on this clause in the specified
OpenMP version. This replaced several individual checks.

Add a testcase for handling map modifiers in a different order, and for
diagnosing an ultimate modifier out of position.



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