[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 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 llvm-commits
mailing list