Skip to content

Set the Fuchsia ABI to the documented minimum#155072

Open
ilovepi wants to merge 1 commit intorust-lang:mainfrom
ilovepi:fuchsia-riscv-driver-default
Open

Set the Fuchsia ABI to the documented minimum#155072
ilovepi wants to merge 1 commit intorust-lang:mainfrom
ilovepi:fuchsia-riscv-driver-default

Conversation

@ilovepi
Copy link
Copy Markdown
Contributor

@ilovepi ilovepi commented Apr 9, 2026

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 9, 2026

These commits modify compiler targets.
(See the Target Tier Policy.)

@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 Apr 9, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 9, 2026

r? @mati865

rustbot has assigned @mati865.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 69 candidates
  • Random selection from 12 candidates

@ilovepi
Copy link
Copy Markdown
Contributor Author

ilovepi commented Apr 9, 2026

@jieyouxu
Copy link
Copy Markdown
Member

jieyouxu commented Apr 10, 2026

(This itself seems fine, note that the linked fuchsia doc is google-internal I think?

EDIT: the link is https://fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v)

@jieyouxu jieyouxu added relnotes Marks issues that should be documented in the release notes of the next release. labels Apr 10, 2026
Copy link
Copy Markdown
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Thanks
r? me @bors r+ rollup

View changes since this review

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors bot commented Apr 10, 2026

📌 Commit a2255aa has been approved by jieyouxu

It is now in the queue for this repository.

@rust-bors rust-bors bot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 10, 2026
@rust-bors rust-bors bot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 10, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Apr 10, 2026
…lt, r=jieyouxu

Set the Fuchsia ABI to the documented minimum

Fuchsia only supports the RVA22 + vector as its minimum ABI for RISC-V.

See: [fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v](https://fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v)[fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v](https://fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v)
rust-bors bot pushed a commit that referenced this pull request Apr 10, 2026
Rollup of 10 pull requests

Successful merges:

 - #154973 (Break a single query cycle in the deadlock handler)
 - #155034 (Implement `GenericTypeVisitable` for some types)
 - #151377 (Fix linker error by resolving regions for main return type obligations)
 - #154677 (hwaddress: automatically add `-Ctarget-feature=+tagged-globals`)
 - #154774 (Add myself as co-maintainer for hexagon-unknown-linux-musl)
 - #155015 (Add tests for three fixed issues (an LLVM crash, an ICE and poor codegen))
 - #155039 (minor follow up to removing soft mode `#[unstable]`)
 - #155043 (Fix if branch in op.rs)
 - #155071 (Deny `#[global_allocator]` + `#[thread_local]`)
 - #155072 (Set the Fuchsia ABI to the documented minimum)
@ilovepi
Copy link
Copy Markdown
Contributor Author

ilovepi commented Apr 10, 2026

(This itself seems fine, note that the linked fuchsia doc is google-internal I think?

EDIT: the link is https://fuchsia.dev/fuchsia-src/contribute/governance/rfcs/0234_riscv_abi_rva22+v)

oh, good catch! thanks for fixing the desciption.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

4 participants