[PATCH] D145057: [clang][ASTImport] Add support for import of empty records
Pavel Kosov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 13 23:54:03 PDT 2023
kpdev42 marked an inline comment as done.
kpdev42 added inline comments.
================
Comment at: clang/include/clang/AST/DeclCXX.h:1171
+ /// attribute
+ void markEmpty() { data().Empty = true; }
----------------
balazske wrote:
> This change looks not related.
The `markEmpty` is still needed and is used in corresponding unit tests. It's gonna be used in `DWARFASTParserClang` to mark records with empty fields as having zero size. Then by copying attributes in `ASTImporter` we prevent this empty mark to be overwritten
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145057/new/
https://reviews.llvm.org/D145057
More information about the cfe-commits
mailing list