sammccall added a comment.

In D114251#3197713 <https://reviews.llvm.org/D114251#3197713>, @davrec wrote:

> One other thought: can we add diagnostic notes using this new information, 
> e.g.

This sounds plausible, though I didn't want to extend the scope of this patch 
further.
I'm also not sure how often this will be signal rather than noise - I would 
expect in a large majority of cases that understanding exactly how the name was 
brought into scope isn't going to be important to understanding or fixing the 
error. I may be missing something, but the example seems a bit contrived.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114251/new/

https://reviews.llvm.org/D114251

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to