Skip to content

Fix build of windows arm64 crossdac - #112553

Merged
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:fix-arm64-crossdac
Feb 16, 2025
Merged

Fix build of windows arm64 crossdac#112553
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:fix-arm64-crossdac

Conversation

@jkotas

Copy link
Copy Markdown
Member

Partial revert of #112256

CopilotAI review requested due to automatic review settings February 14, 2025 05:19

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • src/coreclr/build-runtime.cmd: Language not supported

@ghostghost added the area-Infrastructure-coreclr Only use for closed issues label Feb 14, 2025
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@jkotas

jkotas commented Feb 14, 2025

Copy link
Copy Markdown
MemberAuthor

This is reverting change that broke build of windows arm64 crossdac (the build produces bad binary). Broken crossdac makes it impossible to reasonably investigate windows arm64 crashes that are hit in CI.

If you can fix the problem quickly, feel free to close this PR. If it is going to take more than a day to merge proper fix, I would like to see this revert merged.

@jkotas

Copy link
Copy Markdown
MemberAuthor

cc @dotnet/dotnet-diag

@jkotas

Copy link
Copy Markdown
MemberAuthor

Let's merge this while we discuss the proper fix.

@jkotas
jkotas merged commit 0290a8b into dotnet:mainFeb 16, 2025
@jkotas
jkotas deleted the fix-arm64-crossdac branch February 16, 2025 02:26
grendello added a commit to grendello/runtime that referenced this pull request Feb 18, 2025
* main: (71 commits)
Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20250212.3 (dotnet#112626)
JIT: Unify struct arg morphing (dotnet#112612)
Enable `SA1015`: Closing generic bracket should not be followed by a space (dotnet#112597)
Clean up normalizeLocale for mono browser target (dotnet#112575)
SPMI: Ensure proper zero extension for isObjectImmutable and friends (dotnet#112617)
Quote --version-scripts path (dotnet#112603)
Remove incompatible API from PKCS netstandard2.0 lib
[main] Update dependencies from dotnet/emsdk (dotnet#112393)
Avoid `Unsafe.As` in `RangeCharSearchValues` (dotnet#112606)
Fixed the issue of incorrect return value of PalVirtualAlloc (dotnet#112579)
Fix size used for vectorization check in BitArray (dotnet#111558) (dotnet#111564)
Fix build of windows arm64 crossdac (dotnet#112553)
Simplify `ShuffleTakeIterator.GetCount` (dotnet#112593)
Fix VS div-by-0 in DacEnumerableHashTable code (dotnet#112542)
R2RDump: normalize GC info totalInterruptibleLength (dotnet#112003)
Fix alignment padding and add test for saving managed resources (dotnet#110915)
Adds `ccmp` logic into emitter backend. (dotnet#112153)
Disable AVX10.2 by default (dotnet#112572)
Outbox AesGcm in to Microsoft.Bcl.Cryptography
Make test `IUnknown` conforming (dotnet#112566)
...
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Mar 18, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants

@jkotas@mikem8361