[llvm-branch-commits] [flang] [llvm] [flang][OpenMP] Switch OpenMP version from unsigned to llvm::omp::Version (PR #219822)

Krzysztof Parzyszek via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Sep 1 07:16:34 PDT 2026


================
@@ -1958,8 +1960,9 @@ void CompilerInvocation::setDefaultPredefinitions() {
   }
   if (frontendOptions.features.IsEnabled(
           Fortran::common::LanguageFeature::OpenMP)) {
-    Fortran::common::setOpenMPMacro(getLangOpts().OpenMPVersion,
-                                    fortranOptions.predefinitions);
+    Fortran::common::setOpenMPMacro(
+        static_cast<unsigned>(getLangOpts().getOpenMP()),
+        fortranOptions.predefinitions);
----------------
kparzysz wrote:

I wanted to hide the underlying `OpenMPVersion` member in LangOptions.  I would have changed its type directly to llvm::omp::Version, but its declaration is autogenerated from a .def file and it doesn't allow custom types.

https://github.com/llvm/llvm-project/pull/219822


More information about the llvm-branch-commits mailing list