kadircet accepted this revision. kadircet added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/lib/Sema/SemaDecl.cpp:14995 + DeclFilterCCC<FunctionDecl> CCC{}; + Corrected = CorrectTypo(DeclarationNameInfo(&II, Loc), LookupOrdinaryName, + S, nullptr, CCC, CTK_NonError); ---------------- maybe some comments around importance of the sequencing here? it would probably be nice to have that at a higher level documentation too, but unclear where. as in theory it's not just about the typo correction but also emitting diagnostics while doing so. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115490/new/ https://reviews.llvm.org/D115490 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits