shafik added inline comments.
================
Comment at: lib/AST/ASTStructuralEquivalence.cpp:302
+/// is inspired by ASTContext::mergeFunctionTypes(), we compare calling
+/// conventions bits but must not compare some other bits, e.g. the noreturn
+/// bit.
----------------
This comment is confusing b/c it looks like the noreturn bits are the only one
you are not checking.
================
Comment at: unittests/AST/StructuralEquivalenceTest.cpp:373
+TEST_F(StructuralEquivalenceFunctionTest,
+ FunctionsWithDifferentAttributesButSameTypesShouldBeEqual) {
----------------
Can we get some more tests to be a little more thorough and can we also get a
test where it is expected to to be false as well?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53699/new/
https://reviews.llvm.org/D53699
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
