[Lldb-commits] [PATCH] D29288: Switch std::call_once to llvm::call_once

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

See inlined comment about initialization when llvm::once_flag is a member variable



================
Comment at: include/lldb/Core/Debugger.h:379
   lldb::ListenerSP m_forward_listener_sp;
-  std::once_flag m_clear_once;
+  llvm::once_flag m_clear_once;
 
----------------
Is there a valid default initializer for the llvm::once_flag? If so, we are good to go, if not, we need to either initialize it manually, probably by adding another macro to llvm to do the initialization. Static versions might rely on the compiler zeroing out the memory...


Repository:
  rL LLVM

https://reviews.llvm.org/D29288