[Openmp-commits] [PATCH] D154652: [OpenMP] Move KMP_VERSION printout logic to post-serial-init

[Jonathan Peyton via Phabricator via Openmp-commits]

jlpeyton created this revision.
jlpeyton added a reviewer: tlwilmar.
jlpeyton added a project: OpenMP.
Herald added subscribers: sunshaoce, guansong, yaxunl.
Herald added a project: All.
jlpeyton requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: jplehr, sstefan1.

Get the `KMP_VERSION` printout logic out of environment variable file (`kmp_settings.cpp`) and move to end of serial initialization where `KMP_SETTINGS` and `OMP_DISPLAY_ENV` are.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D154652

Files:
  openmp/runtime/src/kmp_runtime.cpp
  openmp/runtime/src/kmp_settings.cpp


Index: openmp/runtime/src/kmp_settings.cpp
===================================================================
--- openmp/runtime/src/kmp_settings.cpp
+++ openmp/runtime/src/kmp_settings.cpp
@@ -6362,10 +6362,6 @@
 
 #endif /* KMP_AFFINITY_SUPPORTED */
 
-  if (__kmp_version) {
-    __kmp_print_version_1();
-  }
-
   // Post-initialization step: some env. vars need their value's further
   // processing
   if (string != NULL) { // kmp_set_defaults() was called
Index: openmp/runtime/src/kmp_runtime.cpp
===================================================================
--- openmp/runtime/src/kmp_runtime.cpp
+++ openmp/runtime/src/kmp_runtime.cpp
@@ -7315,6 +7315,10 @@
 
   __kmp_init_serial = TRUE;
 
+  if (__kmp_version) {
+    __kmp_print_version_1();
+  }
+
   if (__kmp_settings) {
     __kmp_env_print();
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D154652.537848.patch
Type: text/x-patch
Size: 824 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20230706/921886c7/attachment.bin>