Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

support repr layout options of aligned and packed on structures #915

Closed
philberty opened this issue Feb 9, 2022 · 0 comments · Fixed by #1188
Closed

support repr layout options of aligned and packed on structures #915

philberty opened this issue Feb 9, 2022 · 0 comments · Fixed by #1188

Comments

@philberty
Copy link
Member

https://doc.rust-lang.org/reference/type-layout.html#representations

@philberty philberty added this to the Macro Expansion milestone Feb 9, 2022
@philberty philberty added this to To do in Control Flow 3 Macros via automation Feb 9, 2022
@philberty philberty removed this from To do in Control Flow 3 Macros Mar 16, 2022
@philberty philberty removed this from the Macro Expansion milestone Mar 16, 2022
@dafaust dafaust self-assigned this Apr 20, 2022
@philberty philberty added this to To do in Imports and Visbility via automation Apr 29, 2022
@philberty philberty added this to the Imports and visibility milestone Apr 29, 2022
bors bot added a commit that referenced this issue Apr 29, 2022
1188: Support align and packed repr layout on structs r=dafaust a=dafaust

This is a start at handling the various layout options supported by Rust, beginning with `#[repr(align(N))]` and `#[repr(packed(N))]`, on structs and tuple structs.

There are several other layout options which remain to be supported such as `#[repr(C)]`, `#[repr(transparent)]`, combinations e.g. `#[repr(C, packed(2))]`, as well as layouts on union and enum types.

Fixes: #915 

Co-authored-by: David Faust <david.faust@oracle.com>
@bors bors bot closed this as completed in #1188 Apr 29, 2022
Imports and Visbility automation moved this from To do to Done Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants