eugenis added inline comments.
================
Comment at: compiler-rt/lib/gwp_asan/platform_specific/mutex_posix.cpp:43
+
+Mutex::Mutex() : PImpl(new Impl) {}
+Mutex::~Mutex() { delete PImpl; }
----------------
This is a dependency on libc++ / libstdc++.
I'm not sure about using malloc() inside mutex constructor, either.
We will probably want the mutex to be linker-initialized, too. AFAIK, gwp-asan
would not have any clear initialization entry point, and would need to do this
lazily in malloc, which can there be several of, concurrently.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61923/new/
https://reviews.llvm.org/D61923
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
