Skip to content

Commit a80b6b1

Browse files
authored
Fix running execution tests on x86 (microsoft#7853)
* Updated FindTAEF to use correct names for x86 platforms (WIn32 and X86) * Updated DownloadWarp to pick appropriate version of WARP for the target platform * Updated taef.py to not use /inproc. This allows us to run x86 tests on an x64 machine.
1 parent b6c1914 commit a80b6b1

3 files changed

Lines changed: 20 additions & 12 deletions

File tree

cmake/modules/FindTAEF.cmake

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ set(TAEF_SDK_BIN ${TAEF_INCLUDE_DIR}/../../Runtimes/TAEF)
4444
if ((CMAKE_GENERATOR_PLATFORM STREQUAL "x64") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" STREQUAL "x64"))
4545
set(TAEF_BIN_ARCH "amd64")
4646
set(TAEF_ARCH "x64")
47-
elseif ((CMAKE_GENERATOR_PLATFORM STREQUAL "x86") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" STREQUAL "x86"))
47+
elseif ((CMAKE_GENERATOR_PLATFORM STREQUAL "Win32") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" STREQUAL "X86"))
4848
set(TAEF_BIN_ARCH "x86")
4949
set(TAEF_ARCH "x86")
5050
elseif ((CMAKE_GENERATOR_PLATFORM MATCHES "ARM64.*") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" MATCHES "ARM64.*"))
@@ -53,7 +53,12 @@ elseif ((CMAKE_GENERATOR_PLATFORM MATCHES "ARM64.*") OR ("${CMAKE_C_COMPILER_ARC
5353
elseif ((CMAKE_GENERATOR_PLATFORM MATCHES "ARM.*") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" MATCHES "ARM.*"))
5454
set(TAEF_BIN_ARCH "arm")
5555
set(TAEF_ARCH "arm")
56-
endif((CMAKE_GENERATOR_PLATFORM STREQUAL "x64") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" STREQUAL "x64"))
56+
else ()
57+
message(FATAL_ERROR
58+
"Unable to determine TAEF architecture - CMAKE_GENERATOR_PLATFORM is "
59+
"${CMAKE_GENERATOR_PLATFORM}, CMAKE_C_COMPILER_ARCHITECTURE_ID is "
60+
"${CMAKE_C_COMPILER_ARCHITECTURE_ID}.")
61+
endif()
5762

5863
set (TAEF_ARCH ${TAEF_ARCH} CACHE INTERNAL "arch for taef test")
5964
find_program(TAEF_EXECUTABLE te.exe PATHS

tools/clang/test/taef_exec/DownloadWarp.cmake

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,16 +76,15 @@ endif()
7676

7777
set(WARP_DIR ${CMAKE_BINARY_DIR}/nuget/${WARP_PACKAGE})
7878

79-
if(${CMAKE_SYSTEM_PROCESSOR} STREQUAL "AMD64")
80-
set(WARP_ARCH "x64")
81-
endif()
82-
if(${CMAKE_SYSTEM_PROCESSOR} STREQUAL "X86")
83-
set(WARP_ARCH "win32")
84-
endif()
8579
if(${CMAKE_SYSTEM_PROCESSOR} STREQUAL "ARM64")
8680
set(WARP_ARCH "arm64")
81+
elseif ((CMAKE_GENERATOR_PLATFORM STREQUAL "Win32") OR ("${CMAKE_C_COMPILER_ARCHITECTURE_ID}" STREQUAL "X86"))
82+
set(WARP_ARCH "win32")
83+
else()
84+
set(WARP_ARCH "x64")
8785
endif()
8886

87+
8988
# WARP_DLL is picked up by lit.site.cfg.in so it can be passed as a TAEF runtime
9089
# parameter by lit.cfg
9190
set(WARP_DLL ${WARP_DIR}/build/native/bin/${WARP_ARCH}/d3d10warp.dll)

utils/lit/lit/formats/taef.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -125,10 +125,14 @@ def execute(self, test, litConfig):
125125
if self.select_filter != "":
126126
select_filter = str.format("{} AND {}", select_filter, self.select_filter)
127127

128-
cmd = [self.te, test_dll, '/inproc',
129-
'/select:', select_filter,
130-
'/unicodeOutput:false',
131-
str.format('/outputFolder:{}', self.test_path)]
128+
cmd = [
129+
self.te,
130+
test_dll,
131+
"/select:",
132+
select_filter,
133+
"/unicodeOutput:false",
134+
str.format("/outputFolder:{}", self.test_path),
135+
]
132136
cmd.extend(self.extra_params)
133137

134138
if litConfig.useValgrind:

0 commit comments

Comments
 (0)