[PATCH] D108421: Mark openmp internal global dso_local

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 23 20:31:47 PDT 2021


MaskRay added inline comments.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:2194
+      getOrCreateInternalVariable(KmpCriticalNameTy, Name));
+  if (!GV->isDSOLocal())
+    GV->setDSOLocal(true);
----------------
Can be variable be preemptible on ELF? (i.e. default visibility non-local linkage) If yes, it cannot be marked dso_local in that case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108421



More information about the cfe-commits mailing list