[PATCH] D97191: [clang][NFC] Reorder CXXCtorInitializer members

2021-02-22 Thread Nathan James via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGdaeb70be0bd4: [clang][NFC] Reorder CXXCtorInitializer members (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97191/new/ https://revie

[PATCH] D97191: [clang][NFC] Reorder CXXCtorInitializer members

2021-02-22 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, good catch! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97191/new/ https://reviews.llvm.org/D97191 __

[PATCH] D97191: [clang][NFC] Reorder CXXCtorInitializer members

2021-02-22 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added reviewers: rsmith, aaron.ballman. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Swapping the order of Init and MemberOrEllipsisLocation removes 8 bytes (20%) of padding on 64bit bui