hazohelet added a comment.

In D156604#4572994 <https://reviews.llvm.org/D156604#4572994>, @tbaeder wrote:

> While both of those suggestions are probably improvements (I haven't 
> checked), they also seem out of scope for this patch. This is just adding 
> source ranges. We can improve them later (and add better tests for them at 
> that point).

That makes sense to me.


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

https://reviews.llvm.org/D156604

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

Reply via email to