Fix 1: wrong statement in no_unique_address section#38
Merged
Charliechen114514 merged 1 commit intoJun 10, 2026
Merged
Conversation
Member
|
非常感谢您的反馈,我这边本地也顺手验证(作为典型的本地机器) 验证环境
编译与运行验证使用 PR 中涉及的三个结构体编写了测试程序,编译运行结果如下:
原文的确存在问题原文的断言: static_assert(sizeof(Container) == sizeof(int) + 1); // 可能有 padding实际 这个修改没有显著问题,非常感谢您的批评指正! |
105754f
into
Awesome-Embedded-Learning-Studio:main
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brief: Fix wrong statement about the struct size in no_unique_address section
In the vast majority of compiler implementation, the
struct Containerwill have the padding and be of8bytes in size due to the memory address alignment requirement.Please refer to: https://godbolt.org/z/Gfo7o615q
@Charliechen114514