[PATCH] D61357: SemaOverload: Complete candidates before emitting the error, to ensure diagnostics emitted (or suppressed) during completion don't interfere with the overload notes
David Blaikie via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 30 17:46:44 PDT 2019
dblaikie created this revision.
dblaikie added a reviewer: rsmith.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Because diagnostics and their notes are not connected at the API level,
if the error message for an overload is emitted, then the overload
candidates are completed - if a diagnostic is emitted during that work,
the notes related to overload candidates would be attached to the latter
diagnostic, not the original error. Sort of worse, if the latter
diagnostic was disabled, the notes are disabled.
Repository:
rC Clang
https://reviews.llvm.org/D61357
Files:
include/clang/AST/TemplateName.h
include/clang/Basic/PartialDiagnostic.h
include/clang/Sema/Overload.h
lib/AST/TemplateName.cpp
lib/Sema/SemaCast.cpp
lib/Sema/SemaExprCXX.cpp
lib/Sema/SemaInit.cpp
lib/Sema/SemaOverload.cpp
lib/Sema/SemaStmt.cpp
test/SemaCXX/overload-template.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61357.197491.patch
Type: text/x-patch
Size: 36013 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190501/1db65f11/attachment-0001.bin>
More information about the cfe-commits
mailing list