- User Since
- Nov 10 2019, 5:58 AM (10 w, 2 d)
Mon, Jan 13
Another option that came to my mind is using a BitVector to (recursively) flag bits that are occupied by the fields. This solution would be slightly slower and uses more memory but is probably a lot easier to understand, maintain and more robust than the currently proposed implementation. This would also catch a few additional cases as it could also look inside unions.
Sun, Jan 12
Address (most of the) comments by @aaron.ballman
- remove top-level const on locals
- move declaration into if
- pass TagDecl to diag
- added test for operator void *
- fixed [[no_unique_address]]
- remove assertion that checks for overlapping fields
- in hasPaddingBetweenFields: only do recursive call and add field size to TotalSize if not isZeroSize
- + added tests
Tue, Jan 7
ping @aaron.ballman - any thoughts on the patch?
Thu, Jan 2
Thanks for all the feedback @JonasToth :)
Punctuation in comment
Tests: Split C/C++ tests and add 32/64bit specific test.
Coding guide and better diagnostic message for padding comparison
Mon, Dec 30
ReleaseNotes: move alias after new checks
Sun, Dec 29
Update style based on comments.
Dec 15 2019
Nov 25 2019
Mark comments as Done.
Nov 18 2019
Formatting changes (curly braces, newline at EOF)
Remove incorrect flag from test
Nov 17 2019
Moved check to CERT module.
Added warnings for calling mutating member functions.
Nov 11 2019
Update documentation and release notes.