[PATCH] D37883: cfi: Enable ThinLTO tests on Windows.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 19:06:04 PDT 2017


pcc created this revision.
Herald added subscribers: inglorion, mgorny.

We now avoid using absolute symbols on Windows (https://reviews.llvm.org/D37407 and https://reviews.llvm.org/D37408),
so this should work.

Fixes PR32770.


https://reviews.llvm.org/D37883

Files:
  compiler-rt/test/cfi/CMakeLists.txt


Index: compiler-rt/test/cfi/CMakeLists.txt
===================================================================
--- compiler-rt/test/cfi/CMakeLists.txt
+++ compiler-rt/test/cfi/CMakeLists.txt
@@ -33,8 +33,8 @@
   # FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32741
   add_cfi_test_suites(False False)
 elseif(WIN32)
-  # FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32770
   add_cfi_test_suites(True False)
+  add_cfi_test_suites(True True)
 else()
   if (CFI_SUPPORTED_ARCH)
     add_cfi_test_suites(False False)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37883.115339.patch
Type: text/x-patch
Size: 542 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170915/e9ca1e81/attachment.bin>


More information about the llvm-commits mailing list