[Openmp-commits] [PATCH] D13072: [OpenMP] Enable ThreadSanitizer to check OpenMP programs

[Simone via Openmp-commits]

simoatze added a comment.

In http://reviews.llvm.org/D13072#252137, @hfinkel wrote:

> Thanks for working on this; I'm very excited to see this functionality!
>
> As a general comment, can you always define ANNOTATE_* macros, and just make them empty if TSAN_SUPPORT is not true. What would make things less verbose.


Oh I guess you mean to remove all the TSAN_SUPPORT and just leave the ANNOTATIONS, if TSAN support is not defined it will make them empty, right?


http://reviews.llvm.org/D13072