Skip to content

Conversation

@kparzysz
Copy link
Contributor

No description provided.

@kparzysz kparzysz requested a review from Stylie777 December 19, 2025 14:10
@llvmbot llvmbot added flang:openmp clang:openmp OpenMP related changes to Clang labels Dec 19, 2025
@llvmbot
Copy link
Member

llvmbot commented Dec 19, 2025

@llvm/pr-subscribers-flang-openmp

Author: Krzysztof Parzyszek (kparzysz)

Changes

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

1 Files Affected:

  • (modified) llvm/include/llvm/Frontend/OpenMP/OMP.td (+18-18)
diff --git a/llvm/include/llvm/Frontend/OpenMP/OMP.td b/llvm/include/llvm/Frontend/OpenMP/OMP.td
index 01d0b9af20bad..84f04e92d7746 100644
--- a/llvm/include/llvm/Frontend/OpenMP/OMP.td
+++ b/llvm/include/llvm/Frontend/OpenMP/OMP.td
@@ -112,10 +112,6 @@ def OMPC_CancellationConstructType
   let flangClass = "OmpCancellationConstructTypeClause";
   let skipFlangUnparser = true;
 }
-def OMPC_Contains : Clause<[Spelling<"contains">]> {
-  let clangClass = "OMPContainsClause";
-  let flangClass = "OmpContainsClause";
-}
 def OMPC_Capture : Clause<[Spelling<"capture">]> {
   let clangClass = "OMPCaptureClause";
 }
@@ -131,6 +127,10 @@ def OMPC_Combiner : Clause<[Spelling<"combiner">]> {
 def OMPC_Compare : Clause<[Spelling<"compare">]> {
   let clangClass = "OMPCompareClause";
 }
+def OMPC_Contains : Clause<[Spelling<"contains">]> {
+  let clangClass = "OMPContainsClause";
+  let flangClass = "OmpContainsClause";
+}
 def OMPC_Copyin : Clause<[Spelling<"copyin">]> {
   let clangClass = "OMPCopyinClause";
   let flangClass = "OmpObjectList";
@@ -245,6 +245,9 @@ def OMPC_GraphReset : Clause<[Spelling<"graph_reset">]> {
   let flangClass = "OmpGraphResetClause";
   let isValueOptional = true;
 }
+def OMPC_GroupPrivate : Clause<[Spelling<"groupprivate">]> {
+  let isImplicit = true;
+}
 def OMPC_HasDeviceAddr : Clause<[Spelling<"has_device_addr">]> {
   let clangClass = "OMPHasDeviceAddrClause";
   let flangClass = "OmpObjectList";
@@ -338,12 +341,12 @@ def OMPC_Message : Clause<[Spelling<"message">]> {
 def OMPC_NoOpenMP : Clause<[Spelling<"no_openmp">]> {
   let clangClass = "OMPNoOpenMPClause";
 }
-def OMPC_NoOpenMPRoutines : Clause<[Spelling<"no_openmp_routines">]> {
-  let clangClass = "OMPNoOpenMPRoutinesClause";
-}
 def OMPC_NoOpenMPConstructs : Clause<[Spelling<"no_openmp_constructs">]> {
   let clangClass = "OMPNoOpenMPConstructsClause";
 }
+def OMPC_NoOpenMPRoutines : Clause<[Spelling<"no_openmp_routines">]> {
+  let clangClass = "OMPNoOpenMPRoutinesClause";
+}
 def OMPC_NoParallelism : Clause<[Spelling<"no_parallelism">]> {
   let clangClass = "OMPNoParallelismClause";
 }
@@ -509,6 +512,11 @@ def OMPC_Schedule : Clause<[Spelling<"schedule">]> {
     OMP_SCHEDULE_Default
   ];
 }
+def OMPC_SelfMaps : Clause<[Spelling<"self_maps">]> {
+  let clangClass = "OMPSelfMapsClause";
+  let flangClass = "OmpSelfMapsClause";
+  let isValueOptional = true;
+}
 def OMPC_SeqCst : Clause<[Spelling<"seq_cst">]> {
   let clangClass = "OMPSeqCstClause";
 }
@@ -543,9 +551,6 @@ def OMPC_ThreadLimit : Clause<[Spelling<"thread_limit">]> {
 def OMPC_ThreadPrivate : Clause<[Spelling<"threadprivate">]> {
   let isImplicit = true;
 }
-def OMPC_GroupPrivate : Clause<[Spelling<"groupprivate">]> { 
-  let isImplicit = true;
-}
 def OMPC_Threads : Clause<[Spelling<"threads">]> {
   let clangClass = "OMPThreadsClause";
 }
@@ -571,11 +576,6 @@ def OMPC_UnifiedSharedMemory : Clause<[Spelling<"unified_shared_memory">]> {
   let flangClass = "OmpUnifiedSharedMemoryClause";
   let isValueOptional = true;
 }
-def OMPC_SelfMaps : Clause<[Spelling<"self_maps">]> {
-  let clangClass = "OMPSelfMapsClause";
-  let flangClass = "OmpSelfMapsClause";
-  let isValueOptional = true;
-}
 def OMPC_Uniform : Clause<[Spelling<"uniform">]> {
   let flangClass = "Name";
   let isValueList = true;
@@ -596,9 +596,6 @@ def OMPC_Use : Clause<[Spelling<"use">]> {
   let clangClass = "OMPUseClause";
   let flangClass = "OmpUseClause";
 }
-def OMPC_UsesAllocators : Clause<[Spelling<"uses_allocators">]> {
-  let clangClass = "OMPUsesAllocatorsClause";
-}
 def OMPC_UseDeviceAddr : Clause<[Spelling<"use_device_addr">]> {
   let clangClass = "OMPUseDeviceAddrClause";
   let flangClass = "OmpObjectList";
@@ -607,6 +604,9 @@ def OMPC_UseDevicePtr : Clause<[Spelling<"use_device_ptr">]> {
   let clangClass = "OMPUseDevicePtrClause";
   let flangClass = "OmpObjectList";
 }
+def OMPC_UsesAllocators : Clause<[Spelling<"uses_allocators">]> {
+  let clangClass = "OMPUsesAllocatorsClause";
+}
 def OMPC_Weak : Clause<[Spelling<"weak">]> {
   let clangClass = "OMPWeakClause";
 }

Copy link
Contributor

@Stylie777 Stylie777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @kparzysz

@kparzysz kparzysz merged commit 469e524 into main Dec 19, 2025
10 checks passed
@kparzysz kparzysz deleted the users/kparzysz/d01-alphabetize branch December 19, 2025 15:13
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Dec 19, 2025
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.

4 participants