Skip to content

Conversation

mcfi
Copy link
Contributor

@mcfi mcfi commented Jul 5, 2022

Remove the confusing "Consider a field that has an alignment specifier. Empirically, x64 rounds the size of the struct to the next multiple of the alignment, whereas ARM64 rounds the size of the struct to the next multiple of 8." and make the ARM64EC struct packing rules more clear.

Remove the confusing "Consider a field that has an alignment specifier. Empirically, x64 rounds the size of the struct to the next multiple of the alignment, whereas ARM64 rounds the size of the struct to the next multiple of 8." and make the ARM64EC struct packing rules more clear.
@PRMerger7
Copy link
Contributor

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

@mcfi
Copy link
Contributor Author

mcfi commented Jul 5, 2022

I have a simple grammar mistake I want to correct, one sec

@PRMerger18
Copy link
Contributor

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

@opbld32
Copy link

opbld32 commented Jul 5, 2022

Docs Build status updates of commit 7bb4d9f:

✅ Validation status: passed

File Status Preview URL Details
docs/build/arm64ec-windows-abi-conventions.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld33
Copy link

opbld33 commented Jul 5, 2022

Docs Build status updates of commit 73d6a25:

✅ Validation status: passed

File Status Preview URL Details
docs/build/arm64ec-windows-abi-conventions.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Jul 5, 2022

IMPORTANT: When this content is ready to merge, the reviewer must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@PRMerger19 PRMerger19 added the aq-pr-triaged Tracking label for the PR review team label Jul 5, 2022
@PRMerger14
Copy link
Contributor

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

@opbld31
Copy link

opbld31 commented Jul 5, 2022

Docs Build status updates of commit b5f1c7e:

✅ Validation status: passed

File Status Preview URL Details
docs/build/arm64ec-windows-abi-conventions.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@colin-home
Copy link
Contributor

#sign-off

@colin-home
Copy link
Contributor

Just curious if the build automation extends to automatic merges. Doesn't look like it.

@colin-home colin-home merged commit 0bf682f into MicrosoftDocs:main Jul 5, 2022
@opbld33
Copy link

opbld33 commented Jul 5, 2022

Docs Build status updates of commit b5f1c7e:

✅ Validation status: passed

File Status Preview URL Details
docs/build/arm64ec-windows-abi-conventions.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

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.