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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
