[PATCH] Switch ASan to use ThreadRegistry from sanitizer_common.
Alexey Samsonov
samsonov at google.com
Wed Mar 20 07:57:08 PDT 2013
================
Comment at: lib/asan/asan_interceptors.cc:136
@@ +135,3 @@
+ int detached = 0;
+ pthread_attr_getdetachstate(attr, &detached);
+
----------------
Kostya Serebryany wrote:
> you can call pthread_attr_getdetachstate only iff the original attr != 0
> then you don't need myattr
Done
================
Comment at: lib/asan/asan_report.cc:451
@@ +450,3 @@
+ // recursive reports.
+ ThreadRegistryLock *lock = new(registry_lock_placeholder)
+ ThreadRegistryLock(&asanThreadRegistry());
----------------
Kostya Serebryany wrote:
> OMFG!!
> Can you add sanThreadRegistry()->LockAndNeverUnlock() ?
Done
http://llvm-reviews.chandlerc.com/D555
More information about the llvm-commits
mailing list