Skip to content

[RISC-V] Match integer const select only on different values#120748

Merged
jakobbotsch merged 1 commit intodotnet:mainfrom
tomeksowi:select-const-nulls-fix
Nov 3, 2025
Merged

[RISC-V] Match integer const select only on different values#120748
jakobbotsch merged 1 commit intodotnet:mainfrom
tomeksowi:select-const-nulls-fix

Conversation

@tomeksowi
Copy link
Member

Protect against degenerate cases when both legs are e.g. null ref, without it some of the shift cases in MatchIntConstSelectValues would match.

Part of #84834, cc @dotnet/samsung

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Oct 15, 2025
@github-actions github-actions bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Oct 15, 2025
@dotnet-policy-service
Copy link
Contributor

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

@clamp03 clamp03 added the arch-riscv Related to the RISC-V architecture label Oct 16, 2025
@jakobbotsch
Copy link
Member

/ba-g Dead-letter

@jakobbotsch jakobbotsch merged commit 303f142 into dotnet:main Nov 3, 2025
112 of 114 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-riscv Related to the RISC-V architecture area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI 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