Skip to content

Commit

Permalink
Remove support for SOS win-arm .NET Framework hosting (#3387)
Browse files Browse the repository at this point in the history
* Remove support for SOS win-arm .NET Framework hosting

* Revert "Roll to Windows 2019 image to fix arm cross build (#3376)"

This reverts commit d3a6215.

* PR Feedback

Co-authored-by: Mike McLaughlin <[email protected]>

Co-authored-by: Mike McLaughlin <[email protected]>
  • Loading branch information
hoyosjs and mikem8361 committed Sep 15, 2022
1 parent 5c3c4a4 commit ed71a33
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions eng/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,12 +53,12 @@ jobs:
# Official Build Windows Pool
${{ if and(eq(parameters.osGroup, 'Windows_NT'), ne(variables['System.TeamProject'], 'public')) }}:
name: NetCore1ESPool-Internal
demands: ImageOverride -equals Build.Windows.Amd64.VS2022
demands: ImageOverride -equals windows.vs2022.amd64

# Public Windows Build Pool
${{ if and(eq(parameters.osGroup, 'Windows_NT'), eq(variables['System.TeamProject'], 'public')) }}:
name: NetCore-Public
demands: ImageOverride -equals Build.Windows.Amd64.VS2022.Open
demands: ImageOverride -equals windows.vs2022.amd64.open

${{ if and(ne(parameters.dockerImage, ''), ne(parameters.requiresCapPtraceContainer, 'true')) }}:
container: ${{ parameters.dockerImage }}
Expand Down
4 changes: 2 additions & 2 deletions src/SOS/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ if(CLR_CMAKE_HOST_WIN32)
add_compile_options(/Zl) # omit default library name in .OBJ

add_subdirectory(runcommand)
if(NOT CLR_CMAKE_TARGET_ARCH_ARM64)
if(NOT CLR_CMAKE_TARGET_ARCH_ARM64 AND NOT CLR_CMAKE_TARGET_ARCH_ARM)
add_subdirectory(SOS.UnitTests/Debuggees/DesktopClrHost)
endif()
endif(NOT CLR_CMAKE_TARGET_ARCH_ARM64 AND NOT CLR_CMAKE_TARGET_ARCH_ARM)
endif(CLR_CMAKE_HOST_WIN32)

add_definitions(-D_SECURE_SCL=0)
Expand Down
4 changes: 2 additions & 2 deletions src/SOS/Strike/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -131,11 +131,11 @@ if(WIN32)
ntdll.lib
)

if(NOT CLR_CMAKE_HOST_ARCH_ARM64)
if(NOT CLR_CMAKE_HOST_ARCH_ARM64 AND NOT CLR_CMAKE_HOST_ARCH_ARM)
list(APPEND
SOS_LIBRARY
mscoree.lib)
endif(NOT CLR_CMAKE_HOST_ARCH_ARM64)
endif(NOT CLR_CMAKE_HOST_ARCH_ARM64 AND NOT CLR_CMAKE_HOST_ARCH_ARM)

else(WIN32)
add_definitions(-DFEATURE_ENABLE_HARDWARE_EXCEPTIONS)
Expand Down
4 changes: 2 additions & 2 deletions src/SOS/extensions/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@ set(SOURCES
extensions.cpp
)

if(WIN32 AND NOT CLR_CMAKE_HOST_ARCH_ARM64)
if(WIN32 AND NOT CLR_CMAKE_HOST_ARCH_ARM64 AND NOT CLR_CMAKE_HOST_ARCH_ARM)
list(APPEND SOURCES
hostdesktop.cpp
)
endif(WIN32 AND NOT CLR_CMAKE_HOST_ARCH_ARM64)
endif(WIN32 AND NOT CLR_CMAKE_HOST_ARCH_ARM64 AND NOT CLR_CMAKE_HOST_ARCH_ARM)

if(CLR_CMAKE_HOST_UNIX)
add_definitions(-DHOST_UNIX)
Expand Down
4 changes: 2 additions & 2 deletions src/SOS/extensions/hostcoreclr.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
#define DT_LNK 10
#endif

#if !defined(FEATURE_PAL) && !defined(_TARGET_ARM64_)
#if !defined(FEATURE_PAL) && !defined(HOST_ARM64) && !defined(HOST_ARM)
extern HRESULT InitializeDesktopClrHost();
#endif

Expand Down Expand Up @@ -750,7 +750,7 @@ HRESULT InitializeHosting()
return hr;
}
}
#if !defined(FEATURE_PAL) && !defined(_TARGET_ARM64_)
#if !defined(FEATURE_PAL) && !defined(HOST_ARM64) && !defined(HOST_ARM)
hr = InitializeDesktopClrHost();
if (SUCCEEDED(hr))
{
Expand Down

0 comments on commit ed71a33

Please sign in to comment.