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

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 6 09:00:07 PST 2017

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...



More information about the lldb-commits mailing list