Skip to content

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Jul 2, 2019

In particular, test really small allocations, and test alignment both smaller and bigger than MIN_ALIGN.

The original test case is a special case still covered by the new test.

Cc #62251

@rust-highfive
Copy link
Contributor

r? @kennytm

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 2, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Jul 2, 2019

Oh, this test actually fails locally 😂

It needs #62296 to work. I will fold this PR into that.

@RalfJung RalfJung closed this Jul 2, 2019
@RalfJung RalfJung deleted the overalign branch July 2, 2019 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants