Skip to content

Conversation

kparzysz
Copy link
Contributor

@kparzysz kparzysz commented May 7, 2025

Add 3.1 and 4.0 as versions. This will make flang's default OpenMP version (3.1) be included in the list.

Add 3.1 and 4.0 as versions. This will make flang's default OpenMP
version (3.1) be included in the list.
@llvmbot llvmbot added flang:openmp clang:openmp OpenMP related changes to Clang labels May 7, 2025
@llvmbot
Copy link
Member

llvmbot commented May 7, 2025

@llvm/pr-subscribers-flang-openmp

Author: Krzysztof Parzyszek (kparzysz)

Changes

Add 3.1 and 4.0 as versions. This will make flang's default OpenMP version (3.1) be included in the list.


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

1 Files Affected:

  • (modified) llvm/lib/Frontend/OpenMP/OMP.cpp (+1-1)
diff --git a/llvm/lib/Frontend/OpenMP/OMP.cpp b/llvm/lib/Frontend/OpenMP/OMP.cpp
index 2792dc4281015..555e2a61e411e 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[]{45, 50, 51, 52, 60};
+  static unsigned Versions[]{31, 40, 45, 50, 51, 52, 60};
   return Versions;
 }
 

@kparzysz
Copy link
Contributor Author

kparzysz commented May 7, 2025

Typo in branch name

@kparzysz kparzysz closed this May 7, 2025
@kparzysz kparzysz deleted the users/kparzusz/spr/v01-omp-version branch May 7, 2025 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:openmp OpenMP related changes to Clang flang:openmp
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants