Skip to content

Identify riscv64 as valid arch - #37994

Merged
akoeplinger merged 5 commits into
dotnet:mainfrom
am11:feature/arch-port/riscv64
Jan 19, 2024
Merged

Identify riscv64 as valid arch#37994
akoeplinger merged 5 commits into
dotnet:mainfrom
am11:feature/arch-port/riscv64

Conversation

@am11

@am11 am11 commented Jan 14, 2024

Copy link
Copy Markdown
Member

This is to enable riscv64's R2R bring-up in SDK.

@ghost ghost added Area-Infrastructure untriaged Request triage from a team member labels Jan 14, 2024
@am11

am11 commented Jan 15, 2024

Copy link
Copy Markdown
Member Author

cc @marcpopMSFT, @akoeplinger

This and dotnet/installer#18247 are enabler PRs to resolve chicken-egg for R2R-on-new-arch. The results would be observed when these are picked up by the runtime via SDK update and dotnet/runtime#96941 is reexamined with cross-build. If these look harmless, please merge. Thanks! :)

Comment thread src/Containers/Microsoft.NET.Build.Containers/Registry/Registry.cs Outdated
Comment thread src/Tasks/Microsoft.NET.Build.Tasks/ResolveReadyToRunCompilers.cs
@am11

am11 commented Jan 18, 2024

Copy link
Copy Markdown
Member Author

@akoeplinger, does this looks good for merging?

@akoeplinger akoeplinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM with one comment

</PropertyGroup>
<PropertyGroup Condition=" '$(_PlatformWithoutConfigurationInference)' == 'riscv64' ">
<PlatformTarget Condition=" '$(PlatformTarget)' == '' ">riscv64</PlatformTarget>
</PropertyGroup>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd remove the PlatformTarget here too.

@akoeplinger
akoeplinger merged commit 734bbd9 into dotnet:main Jan 19, 2024
@am11
am11 deleted the feature/arch-port/riscv64 branch January 19, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Infrastructure untriaged Request triage from a team member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants