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.

Build Status

Buildable 10265
Build 10265: arc lint + arc unit

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/

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