[PATCH] D32440: [scudo] Move thread local variables into their own files

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 24 14:18:08 PDT 2017


cryptoad marked an inline comment as done.
cryptoad added inline comments.


================
Comment at: lib/scudo/scudo_tls_linux.cpp:30
+static thread_local bool ThreadInited = false;
+static thread_local bool ThreadTornDown = false;
+static thread_local ScudoThreadContext ThreadLocalContext;
----------------
alekseyshl wrote:
> Why those are two separate flags (some combinations are invalid) and not an enum State field in the ThreadLocalContext?
That is a valid point. I'm going to change it.


https://reviews.llvm.org/D32440





More information about the llvm-commits mailing list