[compiler-rt] r276440 - [sanitizers] Enable sanitizer_common tests on Win64

[Reid Kleckner via llvm-commits]

This broke the Win64 self-host check-all bot:
http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/14108

Hopefully it was fixed by disabling the OOM tests in the followup. Those
tests pass locally for me, but I have more swap and RAM. If there are still
issues, please disable the failing tests. I'm getting on a plane so I can't
do it myself.

On Fri, Jul 22, 2016 at 1:14 PM, Reid Kleckner via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: rnk
> Date: Fri Jul 22 12:14:31 2016
> New Revision: 276440
>
> URL: http://llvm.org/viewvc/llvm-project?rev=276440&view=rev
> Log:
> [sanitizers] Enable sanitizer_common tests on Win64
>
> Modified:
>     compiler-rt/trunk/test/sanitizer_common/CMakeLists.txt
>
> Modified: compiler-rt/trunk/test/sanitizer_common/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/sanitizer_common/CMakeLists.txt?rev=276440&r1=276439&r2=276440&view=diff
>
> ==============================================================================
> --- compiler-rt/trunk/test/sanitizer_common/CMakeLists.txt (original)
> +++ compiler-rt/trunk/test/sanitizer_common/CMakeLists.txt Fri Jul 22
> 12:14:31 2016
> @@ -53,9 +53,7 @@ if(COMPILER_RT_INCLUDE_TESTS)
>    list(APPEND SANITIZER_COMMON_TEST_DEPS SanitizerUnitTests)
>  endif()
>
> -# FIXME: Re-enable on 64-bit Windows.
> -if(SANITIZER_COMMON_TESTSUITES AND
> -    (NOT OS_NAME MATCHES "Windows" OR CMAKE_SIZEOF_VOID_P EQUAL 4))
> +if(SANITIZER_COMMON_TESTSUITES)
>    add_lit_testsuite(check-sanitizer "Running sanitizer_common tests"
>      ${SANITIZER_COMMON_TESTSUITES}
>      DEPENDS ${SANITIZER_COMMON_TEST_DEPS})
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160722/93e440aa/attachment.html>