Skip to content

Document MSVC doesn't straddle bit-fields #3376

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

Merged
merged 2 commits into from
Sep 20, 2021

Conversation

workingjubilee
Copy link
Contributor

This PR documents existing behavior in the MSVC toolset.

@PRMerger15
Copy link
Contributor

@workingjubilee : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Contributor

#label:"aq-pr-triaged"

@PRMerger7 PRMerger7 added the aq-pr-triaged Tracking label for the PR review team label Sep 20, 2021
@PRMerger13
Copy link
Contributor

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@workingjubilee
Thanks for the contribution. We mention this implementation-specific behavior in a very oblique way in Padding and alignment of structure members, and in the Bitfields section of x64 software conventions (though the compiler behavior isn't x64-specific). Those pages aren't necessarily where people would go to find this out, so this seems a good place to add that info. I've modified your content to follow our voice and internationalization guidelines.

@colin-home colin-home merged commit 8c25f7f into MicrosoftDocs:master Sep 20, 2021
@workingjubilee
Copy link
Contributor Author

Ah, I missed that. Thank you, cross-referencing it does seem appropriate.

@workingjubilee workingjubilee deleted the patch-2 branch September 21, 2021 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants