[PATCH] D129117: ManagedStatic: eliminate uses for cl::opt in the llvm directory

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 14:19:01 PDT 2022


efriedma added inline comments.


================
Comment at: llvm/lib/Support/Timer.cpp:47
-// causing havoc to ensue.  We "fix" this by creating the string the first time
-// it is needed and never destroying it.
-static ManagedStatic<std::string> LibSupportInfoOutputFilename;
----------------
Is the problem described by this comment not an issue anymore?  I guess the timing of the call to initTimerOptions is more predictable since we aren't using a global variable, but is it guaranteed to happen before the StatisticInfo constructor runs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129117



More information about the llvm-commits mailing list