rsmith added a comment.

I'm not entirely happy with this approach; there are lots of places where we 
perform lookups that are incidental and shouldn't result in an error if we end 
up looking outside the class, and I'm not sure I've marked them all as 
"synthetic". That said, I've tested this over a fairly substantial amount of 
code and all the issues it reported were true positives.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80977



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D80977: D... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D809... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D809... Eli Friedman via Phabricator via cfe-commits

Reply via email to