On Wed, Jul 12, 2017 at 03:45:46PM -0700, Adam Buchbinder wrote: > On Wed, Jul 12, 2017 at 2:51 PM, David Sterba <dste...@suse.cz> wrote: > > > > On Wed, Jul 12, 2017 at 01:04:49PM -0700, Adam Buchbinder wrote: > > > Tested with clang-3.9. > > > > > > Signed-off-by: Adam Buchbinder <abuchbin...@google.com> > > > --- > > > Makefile | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/Makefile b/Makefile > > > index 81598df..8948301 100644 > > > --- a/Makefile > > > +++ b/Makefile > > > @@ -17,6 +17,7 @@ > > > # abort - call abort() on first error (dumps core) > > > # all - shortcut for all of the above > > > # asan - enable address sanitizer compiler feature > > > +# tsan - enable thread sanitizer compiler feature > > > # ubsan - undefined behaviour sanitizer compiler > feature > > > # bcheck - extended build checks > > > # W=123 build with warnings (default: off) > > > @@ -157,6 +158,11 @@ ifneq (,$(findstring asan,$(D))) > > > DEBUG_CFLAGS_INTERNAL += -fsanitize=address > > > endif > > > > > > +ifneq (,$(findstring tsan,$(D))) > > > + DEBUG_CFLAGS_INTERNAL += -fsanitize=thread -fPIE > > > + LD_FLAGS += -fsanitize=thread -ltsan -pie > > > > Why do you need to set PIE here? Is is necessary for tsan? > > Yes; see https://clang.llvm.org/docs/ThreadSanitizer.html: > "Non-position-independent executables are not supported."
Thanks, patch applied. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
