arphaman added a comment.

In https://reviews.llvm.org/D25520#569389, @akyrtzi wrote:

> >   What do you think of the following possible priority heuristic
>
> SGTM.
>
> Changes LGTM. I'd also recommend that as a follow-up patch it would be great 
> to extend the setter completion to variables as well (global variables, 
> fields, ivars, etc.)


Thanks, I will commit this as it is. I will post a follow-up patch that uses 
the priority heuristic that I described above, and will work on related patches 
that you mentioned as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D25520



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

Reply via email to