[PATCH] [ASan tests] Add Windows-specific flags to lib/asan/tests/CMakeLists.txt

Timur Iskhodzhanov timurrrr at google.com
Tue May 13 03:44:10 PDT 2014


Landed everything except for one questionable if() in r208682.

================
Comment at: lib/asan/tests/CMakeLists.txt:77
@@ -50,2 +76,3 @@
 # Unit tests require libstdc++.
-set(ASAN_UNITTEST_COMMON_LINKFLAGS -lstdc++)
+if(NOT MSVC)
+  set(ASAN_UNITTEST_COMMON_LINKFLAGS -lstdc++)
----------------
Alexey Samsonov wrote:
> Why do you need this? If there is no libstdc++ on Windows, you should instead detect this in the build system and introduce
> COMPILER_RT_HAS_LIBSTDCXX like we do for libpthread and libdl.
We had an offline discussion here and wonder why doesn't the driver itself pass this flag?

We also have to do `if(NOT MSVC)` for the recently-introduced `--driver=g++` flag here anyways, so not clear if the extra variable is worth it.

(we can probably discuss that on the commit email)

http://reviews.llvm.org/D3720






More information about the llvm-commits mailing list