[Lldb-commits] [PATCH] D30702: Fix remaining threading issues in Log.h

[Zachary Turner via Phabricator via lldb-commits]

zturner accepted this revision.
zturner added inline comments.


================
Comment at: source/Utility/Log.cpp:82
+  if (mask | flags) {
+    m_options.store(options, std::memory_order_release);
+    m_stream_sp = stream_sp;
----------------
Might as well use `memory_order_relaxed` here.


================
Comment at: unittests/Utility/LogTest.cpp:16
 #include "llvm/Support/Threading.h"
+#include <future>
 #include <thread>
----------------
Is this included needed?


https://reviews.llvm.org/D30702