[PATCH] D141347: Remove the ThreadLocal template from LLVM.

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 20:00:05 PST 2023


majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Support/CrashRecoveryContext.cpp:23
 struct CrashRecoveryContextImpl;
-
-sys::ThreadLocal<const CrashRecoveryContextImpl> &getCurrentContext() {
-  static sys::ThreadLocal<const CrashRecoveryContextImpl> CurrentContext;
-  return CurrentContext;
-}
+LLVM_THREAD_LOCAL static const  CrashRecoveryContextImpl *CurrentContext;
 
----------------
nit: Extra space after const


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141347



More information about the llvm-commits mailing list