[PATCH] D39151: [tsan] Fix warnings in tsan_interceptors.cc from expansion of variadic macros

[Francis Ricci via Phabricator via llvm-commits]

fjricci added inline comments.


================
Comment at: lib/tsan/rtl/tsan_interceptors.h:59
+  if (!thr->is_inited || thr->ignore_interceptors || thr->in_ignored_lib) \
+    return REAL(func)(__VA_ARGS__);                                       \
+  /**/
----------------
xiaobai wrote:
> fjricci wrote:
> > Is it possible to solve this with `##__VA_ARGS__` instead? Not super familiar with how all of this stuff works though.
> Yes, that is one way to solve this. Unfortunately, that is a GNU extension, and is not considered portable. My solution should work on any platform correctly.
Looks like clang supports it, and I believe that we don't support tsan on windows (so don't need to worry about MSVC). But I'll defer to other reviewers on the best way.


https://reviews.llvm.org/D39151