[clang] [Clang] Clarify diagnostic notes for implicitly generated deduction guides (PR #96084)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 20 05:11:00 PDT 2024
================
@@ -12114,6 +12115,35 @@ static void NoteFunctionCandidate(Sema &S, OverloadCandidate *Cand,
return;
}
+ // If this is an implicit deduction guide against a non-user-defined
+ // constructor, add a note for it. These deduction guides nor their
+ // corresponding constructors are not explicitly spelled in the source code,
+ // and simply producing a deduction failure note around the heading of the
+ // enclosing RecordDecl is confusing.
+ //
+ // We prefer adding such notes at the end of the last deduction failure
+ // reason because duplicate code snippets appearing in the diagnostic
+ // are likely becoming noisy.
----------------
Sirraide wrote:
```suggestion
// If this is an implicit deduction guide against an implicitly defined
// constructor, add a note for it. Neither these deduction guides nor their
// corresponding constructors are explicitly spelled in the source code,
// and simply producing a deduction failure note around the heading of the
// enclosing RecordDecl would be confusing.
//
// We prefer adding such notes at the end of the last deduction failure
// reason because duplicate code snippets appearing in the diagnostic
// would likely become noisy.
```
Just a few minor changes here too
https://github.com/llvm/llvm-project/pull/96084
More information about the cfe-commits
mailing list