vsapsai added a comment. Thanks for the review!
================ Comment at: clang/lib/Serialization/ASTReader.cpp:10020 + auto GetMismatchedDeclLoc = [](const NamedDecl *Container, + ODRMismatchDecl DiffType, const Decl *D) { + SourceLocation Loc; ---------------- ChuanqiXu wrote: > DiffType is not necessary here, right? Unfortunately, it is necessary. A little bit later we have ``` if (DiffType == EndOfClass && Tag) ``` where we use `DiffType`. By the way, the implementation isn't really correct for `EndOfClass` and non-TagDecl as `Loc` remains invalid. But that's a separate issue, I've decided to keep the current change to NFC. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128489/new/ https://reviews.llvm.org/D128489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits