Skip to content

Conversation

CaiWeiran
Copy link

This PR addresses and resolves several test failures that occurred when running ./x test on the RISC-V architecture. The issues were due to platform-specific behavior, ABI differences, or code generation inconsistencies specific to RISC-V.

The following tests have been fixed:

  • assembly/dwarf-mixed-versions-lto.rs
  • codegen/const-vector.rs
  • codegen/enum/enum-aggregate.rs
  • codegen/simd/extract-insert-dyn.rs
  • codegen/transmute-scalar.rs
  • codegen/uninhabited-transparent-return-abi.rs

All changes have been tested locally with ./x test on a RISC-V target and now pass as expected.

Notes:

  • These fixes are scoped specifically to enable full test suite compliance for RISC-V targets.
  • No changes were made that affect other architectures.

@rustbot
Copy link
Collaborator

rustbot commented Jul 9, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 9, 2025
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jul 9, 2025

☔ The latest upstream changes (presumably #143502) made this pull request unmergeable. Please resolve the merge conflicts.

@CaiWeiran CaiWeiran closed this Jul 11, 2025
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 11, 2025
@CaiWeiran CaiWeiran deleted the test branch July 14, 2025 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants