Skip to content

Align RV64's cordbregisterset with LA64 and arm64#129854

Open
am11 wants to merge 10 commits into
dotnet:mainfrom
am11:patch-54
Open

Align RV64's cordbregisterset with LA64 and arm64#129854
am11 wants to merge 10 commits into
dotnet:mainfrom
am11:patch-54

Conversation

@am11

@am11 am11 commented Jun 25, 2026

Copy link
Copy Markdown
Member

Follow-up on #129671.

@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Jun 25, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @steveisok, @tommcdon, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

@am11 am11 added arch-loongarch64 arch-riscv Related to the RISC-V architecture labels Jun 25, 2026
Comment thread src/coreclr/debug/di/rsthread.cpp Outdated
// output: the double corresponding to the ULONGLONG input value
LEAF_ENTRY FPFillR8, .TEXT
ld.d $a0, $a0, 0
fld.d $fa0, $a0, 0

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@shushanhf, it looks like this was a bug.

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.

Agree with you.
Thanks!

Comment thread src/coreclr/debug/di/rsthread.cpp Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch-loongarch64 arch-riscv Related to the RISC-V architecture area-Diagnostics-coreclr community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants