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
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
