Skip to content

Commit

Permalink
fix packing example in PackedSyncPtrTest
Browse files Browse the repository at this point in the history
Summary: Latest compilers may require packing both the containing `struct` and the individual field of packable type in the `struct`.

Reviewed By: ckwalsh

Differential Revision: D62906425

fbshipit-source-id: c49d8c4727f9c9aadf96cce64d392abac6f201da
  • Loading branch information
yfeldblum authored and facebook-github-bot committed Sep 18, 2024
1 parent f67b93c commit be81e0c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion folly/test/PackedSyncPtrTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ namespace {
// PackedSyncPtr is a POD struct on gcc.
FOLLY_PACK_PUSH
struct ignore {
PackedSyncPtr<int> foo;
FOLLY_PACK_ATTR PackedSyncPtr<int> foo;
char c;
} FOLLY_PACK_ATTR;
FOLLY_PACK_POP
Expand Down

0 comments on commit be81e0c

Please sign in to comment.