[Frontend][OpenMP] Add 6.1 as a valid OpenMP version

[kparzysz]

No description provided.

[llvmbot]

@llvm/pr-subscribers-flang-driver

@llvm/pr-subscribers-flang-openmp

Author: Krzysztof Parzyszek (kparzysz)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/153628.diff

2 Files Affected:

• (modified) flang/test/Driver/fopenmp-version.F90 (+1-1)
• (modified) llvm/lib/Frontend/OpenMP/OMP.cpp (+1-1)

diff --git a/flang/test/Driver/fopenmp-version.F90 b/flang/test/Driver/fopenmp-version.F90 index c2866561461b7..48b6ddca772c3 100644 --- a/flang/test/Driver/fopenmp-version.F90 +++ b/flang/test/Driver/fopenmp-version.F90 @@ -22,4 +22,4 @@ !RUN: not %flang -c -fopenmp -fopenmp-version=29 %s 2>&1 | FileCheck --check-prefix=ERR-BAD %s -!ERR-BAD: error: '29' is not a valid OpenMP version in '-fopenmp-version=29', valid versions are 31, 40, 45, 50, 51, 52, 60 +!ERR-BAD: error: '29' is not a valid OpenMP version in '-fopenmp-version=29', valid versions are 31, 40, 45, 50, 51, 52, 60, 61 diff --git a/llvm/lib/Frontend/OpenMP/OMP.cpp b/llvm/lib/Frontend/OpenMP/OMP.cpp index 555e2a61e411e..9e625b809de9e 100644 --- a/llvm/lib/Frontend/OpenMP/OMP.cpp +++ b/llvm/lib/Frontend/OpenMP/OMP.cpp @@ -190,7 +190,7 @@ bool isCombinedConstruct(Directive D) { } ArrayRef<unsigned> getOpenMPVersions() { - static unsigned Versions[]{31, 40, 45, 50, 51, 52, 60}; + static unsigned Versions[]{31, 40, 45, 50, 51, 52, 60, 61}; return Versions; } 

[mjklemm]

LGTM

[mjklemm]

Can you please emit a warning when 6.1 is being selected?

[kparzysz]

Sure, give me a few minutes...

[kparzysz]

The warning is The specification for OpenMP 61 is still under development; the behavior of new features is subject to change

[mjklemm]

LGTM if the wording nits are addressed.