assorted cleanup/fixes #7842
assorted cleanup/fixes #7842
Conversation
There was a problem hiding this comment.
I'd r+, but I'm not sure what this was used for in the first place. Could you explain why this can be removed?
There was a problem hiding this comment.
It can be removed because only managed and managed-unique boxes actually ever have that field used. I don't know why we were bothering to set this on stack closures, but it's not unlike the mostly unused (and now removed) headers on exchange allocations.
There was a problem hiding this comment.
Does this mean that the representation of a & closure actually has all of the headers of an @ allocation?
There was a problem hiding this comment.
Yeah, all closures have a 4-word header. All of them need the tydesc field though, but the other 3 fields are a waste on ~fn/&fn.
the only part of the header ~fn and &fn should be touching right now is the tydesc
…earth update most tests to 2021 edition Some tests would no longer work at all, so I added `edition:2015` or `edition:2018` to them. Notably 2021 panics are not yet detected correctly. Once ready, this closes rust-lang#7842. --- changelog: none
3 participants
No description provided.