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