Skip to content

Fix tests for C-unwind ABI changes#1868

Merged
bors merged 1 commit into
masterfrom
unknown repository
Aug 7, 2021
Merged

Fix tests for C-unwind ABI changes#1868
bors merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Aug 5, 2021

Copy link
Copy Markdown

@RalfJung

RalfJung commented Aug 6, 2021

Copy link
Copy Markdown
Member

Blocked on rust-lang/rust#87787 now.

I don't understand, that issue just says "Miri doesn't work"... so what is this blocked on?

EDIT: Oh, 87778 != 87787.

Comment thread tests/compile-fail/function_calls/exported_symbol_bad_unwind2.rs
@ghost
ghost marked this pull request as ready for review August 7, 2021 08:18
@RalfJung

RalfJung commented Aug 7, 2021

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

📌 Commit af7eb36 has been approved by RalfJung

@RalfJung

RalfJung commented Aug 7, 2021

Copy link
Copy Markdown
Member

@bors r+ retry

@bors

bors commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

💡 This pull request was already approved, no need to approve it again.

@bors

bors commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

📌 Commit af7eb36 has been approved by RalfJung

@RalfJung

RalfJung commented Aug 7, 2021

Copy link
Copy Markdown
Member

Bors seems to be broken.

@bors

bors commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

⌛ Testing commit af7eb36 with merge 042cbf1...

@bors

bors commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 042cbf1 to master...

@bors
bors merged commit 042cbf1 into rust-lang:master Aug 7, 2021
@ghost
ghost deleted the rustup branch August 15, 2021 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants