Skip to content

Commit

Permalink
Disable stack depth tracking on Windows.
Browse files Browse the repository at this point in the history
Windows doesn't support the tls_model attribute.

llvm-svn: 312032
  • Loading branch information
morehouse committed Aug 29, 2017
1 parent 154411e commit c29c2c9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions clang/lib/Driver/SanitizerArgs.cpp
Expand Up @@ -315,8 +315,8 @@ SanitizerArgs::SanitizerArgs(const ToolChain &TC,
if (Add & FuzzerNoLink) {
CoverageFeatures |= CoverageTracePCGuard | CoverageIndirCall |
CoverageTraceCmp | CoveragePCTable;
// Due to TLS differences, stack depth tracking is disabled on Mac.
if (!TC.getTriple().isOSDarwin())
// Due to TLS differences, stack depth tracking is disabled on Mac/Win.
if (!TC.getTriple().isOSDarwin() && !TC.getTriple().isOSWindows())
CoverageFeatures |= CoverageStackDepth;
}

Expand Down
2 changes: 1 addition & 1 deletion compiler-rt/lib/sanitizer_common/sanitizer_internal_defs.h
Expand Up @@ -36,7 +36,7 @@
#endif

// Mac handles TLS differently
#if SANITIZER_MAC
#if SANITIZER_MAC || SANITIZER_WINDOWS
# define SANITIZER_TLS_INITIAL_EXEC_ATTRIBUTE
#else
# define SANITIZER_TLS_INITIAL_EXEC_ATTRIBUTE \
Expand Down

0 comments on commit c29c2c9

Please sign in to comment.