bruno marked an inline comment as done.
bruno added inline comments.
================
Comment at: clang/lib/Serialization/ASTReader.cpp:11007
+ // Issue any pending ODR-failure diagnostics.
+ for (auto &Merge : RecordOdrMergeFailures) {
----------------
rtrieu wrote:
> Is this just a copy of the other loop? That's a lot of code duplication.
There's a lot of copy but a lot of CXXRecord specific logic got trimmed out and
there are small differences in the approach, so a direct refactor of it won't
work, but there's certainly more small pieces that can be factored out, will
follow up with that.
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
