[clang] [Clang] Clarify diagnostic notes for implicitly generated deduction guides (PR #96084)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 19 09:20:44 PDT 2024
Sirraide wrote:
I will say, one worry that I do have is that this would end up issuing... a lot of notes for a single error:
![image](https://github.com/llvm/llvm-project/assets/74590115/3f517fb4-a00d-4093-b690-c2e2e1f49380)
C++ error messages already have a reputation of being rather long; I’m candidly doubtful as to whether this would help most users too much...
Perhaps an approach more similar to what we display for ambiguous cast paths would be better (https://godbolt.org/z/zT1rW7zcv):
![image](https://github.com/llvm/llvm-project/assets/74590115/8c4f5ba7-eec8-4c03-a026-7bfd23e01d6d)
i.e. list each implicit deduction guide we tried in a single note; I personally at leas think that using indentation and formatting a bit more rather than emitting a separate note for everything we might want to add to an error might go a long way in aiding legibility, but that’s just my opinion ;Þ
https://github.com/llvm/llvm-project/pull/96084
More information about the cfe-commits
mailing list