[PATCH] D103119: [scudo] Get rid of initLinkerInitialized

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 25 15:17:44 PDT 2021


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

Yay!



================
Comment at: compiler-rt/lib/scudo/standalone/stats.h:62
-    LocalStats::init();
-    Mutex.init();
-    StatsList = {};
----------------
Can you also delete `HybridMutex::init()`?


================
Comment at: compiler-rt/lib/scudo/standalone/tests/tsd_test.cpp:45
+  bool Initialized = false;
+  ;
   TSDRegistryT TSDRegistry;
----------------
nit: floating `;`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103119



More information about the llvm-commits mailing list