[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 6 23:02:01 PST 2023


kpdev42 marked 2 inline comments as done.
kpdev42 added inline comments.


================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:8161
+    EXPECT_EQ(true, FD->hasAttr<NoUniqueAddressAttr>());
+}
+
----------------
balazske wrote:
> Does this test fail without the changes applied? And does it not fail after (is the "Empty" value copied at import)?
Yes the tests will fail due to above mentioned problem with field attributes


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