[PATCH] D156201: [ASTImporter] Fix corrupted RecordLayout introduced by circular referenced fields
Balázs Kéri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 1 08:29:41 PDT 2023
balazske added a reviewer: aaron.ballman.
balazske added a comment.
The fix looks good, only a person with AST competence should have a look at it because a change in `Expr.h`.
================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:8033
+TEST_P(ASTImporterOptionSpecificTestBase,
+ ImportCirularRefFieldsWithoutCorruptedRecordLayoutCacheTest) {
+ // Import sequence: A => A.b => B => B.f() => ... => UnaryOperator(&) => ...
----------------
A small thing, `ImportCirularRefFieldsWithoutCorruptedRecordLayoutCache` (no `Test` ending) is the usual naming for these tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156201/new/
https://reviews.llvm.org/D156201
More information about the cfe-commits
mailing list