[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