[PATCH] D145057: [clang][ASTImport] Add support for import of empty records

Pavel Kosov via Phabricator via llvm-commits llvm-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 llvm-commits mailing list