[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