move function to header

Author: kparzysz

llvm/include/llvm/Frontend/OpenMP/OMP.h

@@ -33,7 +33,19 @@ bool isCompositeConstruct(Directive D);
 bool isCombinedConstruct(Directive D);
 
 /// Can clause C have an iterator-modifier.
-bool canHaveIterator(Clause C);
+static constexpr inline bool canHaveIterator(Clause C) {
+  // [5.2:67:5]
+  switch (C) {
+  case OMPC_affinity:
+  case OMPC_depend:
+  case OMPC_from:
+  case OMPC_map:
+  case OMPC_to:
+    return true;
+  default:
+    return false;
+  }
+}
 
 /// Create a nicer version of a function name for humans to look at.
 std::string prettifyFunctionName(StringRef FunctionName);

llvm/lib/Frontend/OpenMP/OMP.cpp

@@ -193,20 +193,6 @@ bool isCombinedConstruct(Directive D) {
   return !getLeafConstructs(D).empty() && !isCompositeConstruct(D);
 }
 
-bool canHaveIterator(Clause C) {
-  // [5.2:67:5]
-  switch (C) {
-  case OMPC_affinity:
-  case OMPC_depend:
-  case OMPC_from:
-  case OMPC_map:
-  case OMPC_to:
-    return true;
-  default:
-    return false;
-  }
-}
-
 std::string prettifyFunctionName(StringRef FunctionName) {
   // Internalized functions have the right name, but simply a suffix.
   if (FunctionName.ends_with(".internalized"))