[all-commits] [llvm/llvm-project] 2e023c: [OpenMP] Make all syntactic properties explicit (#...

[Krzysztof Parzyszek via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2e023c8e922d2b03c0cd39c1dbb6e594ab01d856
      https://github.com/llvm/llvm-project/commit/2e023c8e922d2b03c0cd39c1dbb6e594ab01d856
  Author: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
  Date:   2026-09-11 (Fri, 11 Sep 2026)

  Changed paths:
    M llvm/include/llvm/Frontend/OpenMP/OMPDescriptors.h.inc
    M llvm/lib/Frontend/OpenMP/OMPDescriptors.inc

  Log Message:
  -----------
  [OpenMP] Make all syntactic properties explicit (#222375)

For clauses, modifiers, and modifier sets/groups, make sure that either
the syntactic proprty or its inverse is explicitily listed, with the
missing properties added according to the following table.
```
                       |                           Modifier
            Inverse    | Clause       Modifier     group
Property    property   | defaults     defaults     defaults
----------+------------+------------+------------+-----------
required    optional   | optional     optional     optional
unique      repeatable | repeatable   unique       unique
exclusive   compatible | compatible   compatible   compatible
ultimate    free       | free         free         free
```


  Commit: 5603248c95d69e31889f9eca8ce5254c3a2484bd
      https://github.com/llvm/llvm-project/commit/5603248c95d69e31889f9eca8ce5254c3a2484bd
  Author: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
  Date:   2026-09-11 (Fri, 11 Sep 2026)

  Changed paths:
    M flang/include/flang/Semantics/openmp-directive-sets.h
    M flang/include/flang/Semantics/symbol.h
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    M flang/lib/Parser/openmp-parsers.cpp
    M flang/lib/Semantics/check-omp-loop.cpp
    M flang/lib/Semantics/check-omp-structure.cpp
    M flang/lib/Semantics/check-omp-structure.h
    M flang/lib/Semantics/mod-file.cpp
    M flang/lib/Semantics/resolve-directives.cpp
    M flang/lib/Semantics/symbol.cpp
    M llvm/include/llvm/Frontend/OpenMP/OMP.h
    M llvm/include/llvm/Frontend/OpenMP/OMP.td
    M llvm/include/llvm/Frontend/OpenMP/OMPDescriptors.h
    M llvm/unittests/Frontend/EnumSetTest.cpp

  Log Message:
  -----------
  [OpenMP] Rename ClauseSet to Clauses and DirectiveSet to Directives, NFC (#222440)

The ClauseSet name was an alias of an EnumSet of clause ids. However, it
collides with a "clause set" concept in OpenMP, which is a set of
clauses,
but with properties and additional information. To resolve the naming
conflict, rename ClauseSet to Clauses.

Although there is no "directive set" in OpenMP, rename DirectiveSet in
an analogous way for uniformity.


Compare: https://github.com/llvm/llvm-project/compare/11ef37f9b78e...5603248c95d6

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