This is an archive of the discontinued LLVM Phabricator instance.

cfi: Enable ThinLTO tests on Windows.
ClosedPublic

Authored by pcc on Sep 14 2017, 7:06 PM.

Reviewers

rnk
eugenis

Commits

Summary

We now avoid using absolute symbols on Windows (D37407 and D37408),
so this should work.

Fixes PR32770.

lgtm

This revision is now accepted and ready to land.Sep 15 2017, 9:53 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

compiler-rt/

trunk/

test/

cfi/

2 lines

Show All 27 Lines	configure_lit_site_cfg(
)		)
list(APPEND CFI_TESTSUITES ${CMAKE_CURRENT_BINARY_DIR}/Devirt${suffix})		list(APPEND CFI_TESTSUITES ${CMAKE_CURRENT_BINARY_DIR}/Devirt${suffix})
endmacro()		endmacro()

if (APPLE)		if (APPLE)
# FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32741		# FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32741
add_cfi_test_suites(False False)		add_cfi_test_suites(False False)
elseif(WIN32)		elseif(WIN32)
# FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32770
add_cfi_test_suites(True False)		add_cfi_test_suites(True False)
		add_cfi_test_suites(True True)
else()		else()
if (CFI_SUPPORTED_ARCH)		if (CFI_SUPPORTED_ARCH)
add_cfi_test_suites(False False)		add_cfi_test_suites(False False)
add_cfi_test_suites(False True)		add_cfi_test_suites(False True)
if (COMPILER_RT_HAS_LLD)		if (COMPILER_RT_HAS_LLD)
add_cfi_test_suites(True False)		add_cfi_test_suites(True False)
add_cfi_test_suites(True True)		add_cfi_test_suites(True True)
endif()		endif()
▲ Show 20 Lines • Show All 45 Lines • Show Last 20 Lines