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
+      *cast<ClassTemplateDecl>(get<1>(t))->spec_begin();
+  ASSERT_TRUE(Spec0 != nullptr);
+  ASSERT_TRUE(Spec1 != nullptr);
a.sidorin wrote:
> Should we assert for `spec_begin() != spec_end()` instead or within these 
> assertions?
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);
a.sidorin wrote:
> 1. It's better to use more meaningful names than `t`. DeclTuple?
> 2. The space after `(` is redundant.
Renamed `t` to `Decls`.

  rC Clang

cfe-commits mailing list

Reply via email to