[Openmp-commits] [PATCH] D66292: [OpenMP] Change initialization of __kmp_global

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld marked an inline comment as done.
Hahnfeld added inline comments.


================
Comment at: openmp/runtime/src/kmp_global.cpp:434
 KMP_ALIGN_CACHE
-kmp_global_t __kmp_global = {{0}};
+kmp_global_t __kmp_global;
 
----------------
The other solution would be explicitly initializing all struct variables, not sure if that is preferred. Putting `__kmp_global` into the `.bss` section seems ok, it's initialized to zero by the loader.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66292/new/

https://reviews.llvm.org/D66292