[PATCH] D123958: [randstruct] Randomize all elements of a record
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 28 06:17:14 PDT 2022
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Comment at: clang/unittests/AST/RandstructTest.cpp:368
+ int h;
+ char name;
+ } __attribute__((randomize_layout));
> aaron.ballman wrote:
> > Can you add a test where the last field is `char name;` and another one for `char name;` so that we have full coverage there?
> The `name` is above this test. The `name` is due to a copy-paste error (oops). Fixed.
Ah, good call on `name`, I missed that one.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits