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

[Kamil Rytarowski via Phabricator via lldb-commits]

krytarowski added inline comments.


================
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;
 
----------------
clayborg wrote:
> 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...
Yes, there is a valid default initializer for llvm::once_flag.

Related entry in LLVM - D29566


Repository:
  rL LLVM

https://reviews.llvm.org/D29288