ioeric added a comment.

In https://reviews.llvm.org/D40563#939555, @arphaman wrote:

> Could you please add a test?


Any tip on how this should be tested? I couldn't find any existing unit test 
for either SemaCodeComplete or code completion context (under 
`clang/unittests`). It might be possible to set up a unit test framework for 
code completion, but I'm not sure if it's worth to do so since this change 
doesn't change completion behavior.


https://reviews.llvm.org/D40563



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

Reply via email to