johannes added a comment.

In https://reviews.llvm.org/D37001#852442, @arphaman wrote:

> Can you remove `getNodeValue` now or do you still need it?


It is only used in the client, I think it makes sense to move it there, or to 
remove it altogether.
I think I'd keep it in the client for now, to disambiguate things with 
different names in the tests.


https://reviews.llvm.org/D37001



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

Reply via email to