martong added inline comments.
================ Comment at: clang/lib/Serialization/ASTReader.cpp:9286 + false /*UseCanonicalDecls*/); + (void)Ctx.IsEquivalent(D, Canon); + } ---------------- Would it be possible to check the structural non-equivalency with `ODRHash`? I wonder if the `ODRHash` of the two Decls are different in this case or not. `ASTStructuralEquivalency` and `ODRHash` seems to serve a very similar use case, however, AFAIK in the rest of the ASTReader code `ODRHash` is used exclusively. (On a long term, I am thinking about that perhaps `ASTImporter` could use ODRHash instead of ASTStructuralEquivalency.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71734/new/ https://reviews.llvm.org/D71734 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
