Skip to content

Conversation

ilovepi
Copy link
Contributor

@ilovepi ilovepi commented Dec 22, 2022

Fuchsia already uses SCS by default for C/C++ code on ARM hardware. This patch allows SCS to be used for Rust code as well.

@rustbot
Copy link
Collaborator

rustbot commented Dec 22, 2022

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @jackh726 (or someone else) soon.

Please see the contribution instructions for more information.

@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 Dec 22, 2022
@rustbot
Copy link
Collaborator

rustbot commented Dec 22, 2022

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

@ilovepi
Copy link
Contributor Author

ilovepi commented Dec 22, 2022

@tmandry Is there anything else we'd need to enable on the Fuchsia side besides adding -Zsanitizer=shadow-call-stack in our build configs?

@jackh726
Copy link
Member

jackh726 commented Jan 4, 2023

Sorry, don't have the background to review this.

r? compiler

@rustbot rustbot assigned oli-obk and unassigned jackh726 Jan 4, 2023
@bors
Copy link
Collaborator

bors commented Jan 6, 2023

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

Fuchsia already uses SCS by default for C/C++ code on ARM hardware.
This patch allows SCS to be used for Rust code as well.
@oli-obk
Copy link
Contributor

oli-obk commented Jan 9, 2023

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 9, 2023

📌 Commit c5bde06 has been approved by oli-obk

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 9, 2023
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Jan 9, 2023
Enable Shadow Call Stack for Fuchsia on AArch64

Fuchsia already uses SCS by default for C/C++ code on ARM hardware. This patch allows SCS to be used for Rust code as well.
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Jan 9, 2023
Enable Shadow Call Stack for Fuchsia on AArch64

Fuchsia already uses SCS by default for C/C++ code on ARM hardware. This patch allows SCS to be used for Rust code as well.
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 9, 2023
…fee1-dead

Rollup of 10 pull requests

Successful merges:

 - rust-lang#105292 (Change a commit_if_ok call to probe)
 - rust-lang#105655 (Remove invalid case for mutable borrow suggestion)
 - rust-lang#106047 (Fix ui constant tests for big-endian platforms)
 - rust-lang#106061 (Enable Shadow Call Stack for Fuchsia on AArch64)
 - rust-lang#106164 (Move `check_region_obligations_and_report_errors` to `TypeErrCtxt`)
 - rust-lang#106291 (Fix incorrect suggestion for extra `&` in pattern)
 - rust-lang#106389 (Simplify some canonical type alias names)
 - rust-lang#106468 (Use FxIndexSet when updating obligation causes in `adjust_fulfillment_errors_for_expr_obligation`)
 - rust-lang#106549 (Use fmt named parameters in rustc_borrowck)
 - rust-lang#106614 (error-code docs improvements (No. 2))

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit fd75cfe into rust-lang:master Jan 9, 2023
@rustbot rustbot added this to the 1.68.0 milestone Jan 9, 2023
@ilovepi ilovepi deleted the fuchsia-scs branch January 9, 2023 21:15
@tmandry tmandry added the O-fuchsia Operating system: Fuchsia label May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-fuchsia Operating system: Fuchsia 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.

6 participants