martong marked 14 inline comments as done.
martong added a comment.
> Do you plan to enable this functionality for AST import checking?
Yes. We'd like to test the structural equivalency independently from
ASTImporter, because in certain cases it may have faulty behavior. This can be
very handy also when we further extend structural equivalency.
Comment at: unittests/AST/StructuralEquivalenceTest.cpp:119
+ ASSERT_TRUE(Spec0 != nullptr);
+ ASSERT_TRUE(Spec1 != nullptr);
> Should we assert for `spec_begin() != spec_end()` instead or within these
I think that is not needed, because the previous `cast` in line 116 and 118
would assert if `spec_begin() == spec_end()` was true.
Comment at: unittests/AST/StructuralEquivalenceTest.cpp:174
+ auto t = makeNamedDecls( Code0, Code0, Lang_CXX);
> 1. It's better to use more meaningful names than `t`. DeclTuple?
> 2. The space after `(` is redundant.
Renamed `t` to `Decls`.
cfe-commits mailing list