Skip to content

Conversation

RalfJung
Copy link
Member

Also handle get_alloc_extra the same throughout Stacked Borrows. I don't think this unwrap can actually cause an ICE since another part of SB will raise an error before, but still, seems strange to do this inconsistently in retagging vs expose_ptr.

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jul 20, 2022

📌 Commit 96c564d has been approved by RalfJung

It is now in the queue for this repository.

bors added a commit that referenced this pull request Jul 20, 2022
check for extern static size mismatches

Also handle get_alloc_extra the same throughout Stacked Borrows. I don't think this `unwrap` can actually cause an ICE since another part of SB will raise an error before, but still, seems strange to do this inconsistently in retagging vs expose_ptr.
@bors
Copy link
Contributor

bors commented Jul 20, 2022

⌛ Testing commit 96c564d with merge c69d9e6...

@bors
Copy link
Contributor

bors commented Jul 20, 2022

💔 Test failed - checks-actions

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jul 20, 2022

📌 Commit 9f99d10 has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 20, 2022

⌛ Testing commit 9f99d10 with merge e8c4c9a...

@bors
Copy link
Contributor

bors commented Jul 20, 2022

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

@bors bors merged commit e8c4c9a into rust-lang:master Jul 20, 2022
@RalfJung RalfJung deleted the mix branch July 20, 2022 23:02
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.

2 participants