[PATCH] D18277: [tsan] Build TSan dylibs for iOS-style simulators

Chris Bieneman via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 15:05:54 PDT 2016

beanz accepted this revision.
beanz added a comment.
This revision is now accepted and ready to land.

Patch LGTM.

Comment at: cmake/config-ix.cmake:400
@@ -399,3 +399,3 @@
         message(STATUS "${platform} Simulator supported arches: ${DARWIN_${platform}sim_ARCHS}")
           list(APPEND SANITIZER_COMMON_SUPPORTED_OS ${platform}sim)
kubabrecka wrote:
> Unrelated to this patch, but shouldn't this be `if(DARWIN_${platform}sim_ARCHS)`?
Yes, yes it should.


More information about the llvm-commits mailing list