Enable win registry install location for all architectures

[mateoatr]

Follow-up of #53763

Enable the multi-architecture install locations on Windows Registry. The registration mechanism works the same way, we are only lifting the requirement of having to be running on x64/x86 to use the registry.

[ghost]

Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Follow-up of #53763

Enable the multi-architecture install locations on Windows Registry. The registration mechanism works the same way, we are only lifting the requirement of having to be running on x64/x86 to use the registry.

Author:	mateoatr
Assignees:	-
Labels:	area-Host
Milestone:	-

[vitek-karas]

Given that you made basically no changes to existing tests - does it mean that we don't run the tests on Windows ARM32 or ARM64 at all?

Or does it mean that we didn't have any tests validating the current behavior?

[elinor-fung]

Apparently the host tests don't run on arm or arm64 at all (Windows or not):

runtime/eng/pipelines/installer/jobs/base-job.yml

Lines 63 to 69 in fc4a427

	- name: SkipTests
	value: ${{ or(
	not(in(parameters.archType, 'x64', 'x86')),
	eq(parameters.runtimeFlavor, 'mono'),
	eq(parameters.isOfficialBuild, true),
	eq(parameters.crossBuild, true),
	eq(parameters.pgoType, 'PGO')) }}

[mateoatr]

I'll enable them on a separate PR (which should be merged before this one) -- curious if this will cause existing tests to fail.

[elinor-fung]

Fair warning - I expect enabling them may be a bit involved. I think the main reason the arm/arm64 tests are currently skipped is that the hosting tests haven't been set up to send tests to helix, so they just run on the build machine (which works fine for x64/x86).

[elinor-fung]

Apparently the host tests don't run on arm or arm64 at all (Windows or not):

runtime/eng/pipelines/installer/jobs/base-job.yml

Lines 63 to 69 in fc4a427

	- name: SkipTests
	value: ${{ or(
	not(in(parameters.archType, 'x64', 'x86')),
	eq(parameters.runtimeFlavor, 'mono'),
	eq(parameters.isOfficialBuild, true),
	eq(parameters.crossBuild, true),
	eq(parameters.pgoType, 'PGO')) }}

[vitek-karas]

I think we should be able to merge this - assuming we did some manual testing of it.

[agocke]

@mateoatr Is this ready to go?

[mateoatr]

@mateoatr Is this ready to go?

Yup. I did some local validation on a win-arm64 machine: arm apps work as expected, running an x86 apphost works fine as well. The dotnet run scenario is broken, but we are already tracking that and working on a fix in the SDK.

[agocke]

Do we still need to swap the DOTNET_ROOT ordering, or is that already done?

[mateoatr]

Do we still need to swap the DOTNET_ROOT ordering, or is that already done?

Nope. It's already done -- I was having issues with dotnet run but simply running the apphost is good, and the ordering is fine.

[agocke]

/backport to release/6.0-rc1

[github-actions]

Started backporting to release/6.0-rc1: https://github.com/dotnet/runtime/actions/runs/1151559541