[PATCH] D38073: [XRay][compiler-rt] Use pthread for initializing thread-local data
Martin Pelikán via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 2 22:11:09 PDT 2017
pelikan accepted this revision.
pelikan added inline comments.
This revision is now accepted and ready to land.
================
Comment at: lib/xray/xray_fdr_logging_impl.h:180
+ // Ensure that we only actually ever do the pthread initialization once.
+ thread_local bool unused = [] {
+ new (&TLSBuffer) ThreadLocalData();
----------------
I reckon this should get the UNUSED sanitizer_common macro, which expands to `__attribute__((unused))` to stop the compiler whinging about it.
Plus, CodingStandards.rst wants it to be called "Unused" (same above - Key).
================
Comment at: lib/xray/xray_fdr_logging_impl.h:209
+ }();
+ (void)unused;
+
----------------
This line can go away with the UNUSED macro up there.
https://reviews.llvm.org/D38073
More information about the llvm-commits
mailing list