Skip to content

Adding needs-unwind arg to applicable compiler ui tests #101741

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Sep 17, 2022

Conversation

andrewpollack
Copy link
Member

Adding needs-unwind arg to applicable compiler ui tests

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Sep 12, 2022
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 12, 2022
@andrewpollack
Copy link
Member Author

These are compiler ui tests I believe that require unwind (i.e. not panic=abort)

cc. @djkoloski @tmandry

@andrewpollack andrewpollack force-pushed the add-needs-unwind-ui-tests branch from 855278c to 4f18358 Compare September 12, 2022 23:12
@@ -12,6 +12,7 @@
// edition:2018
// ignore-wasm32 issue #62807
// ignore-asmjs issue #62807
// needs-unwind Closures off by one byte due to panic byte flag required
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto again here.

@andrewpollack andrewpollack force-pushed the add-needs-unwind-ui-tests branch from 4f18358 to 34a57a2 Compare September 14, 2022 20:34
@andrewpollack
Copy link
Member Author

Removed tests that require further inspection -- revisiting those in separate PR

Copy link
Contributor

@djkoloski djkoloski left a comment

Choose a reason for hiding this comment

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

LGTM

@tmandry
Copy link
Member

tmandry commented Sep 16, 2022

@bors delegate+ rollup

@bors
Copy link
Collaborator

bors commented Sep 16, 2022

✌️ @andrewpollack can now approve this pull request

@andrewpollack andrewpollack force-pushed the add-needs-unwind-ui-tests branch from ac54767 to 5d27f19 Compare September 16, 2022 19:28
@andrewpollack
Copy link
Member Author

@bors r=tmandry

@bors
Copy link
Collaborator

bors commented Sep 16, 2022

📌 Commit 5d27f19 has been approved by tmandry

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 Sep 16, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 17, 2022
Rollup of 8 pull requests

Successful merges:

 - rust-lang#101340 (Adding Fuchsia zxdb debugging walkthrough to docs)
 - rust-lang#101741 (Adding needs-unwind arg to applicable compiler ui tests)
 - rust-lang#101782 (Update `symbol_mangling` diagnostics migration)
 - rust-lang#101878 (More simple formatting)
 - rust-lang#101898 (Remove some unused CSS rules)
 - rust-lang#101911 (rustdoc: remove no-op CSS on `.source .content`)
 - rust-lang#101914 (rustdoc-json-types: Document that ResolvedPath can also be a union)
 - rust-lang#101921 (Pass --cfg=bootstrap for rustdoc for proc_macro crates)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 217a82a into rust-lang:master Sep 17, 2022
@rustbot rustbot added this to the 1.65.0 milestone Sep 17, 2022
@tmandry tmandry added the O-fuchsia Operating system: Fuchsia label May 24, 2023
# for free to join this conversation on GitHub. Already have an account? # 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.

7 participants