Szelethus added a comment.

Makes sense! I like the summary a lot, and the fact that you added a new debug 
checker. I feel like I'm not yet qualified to give meaningful feedback though, 
but if you are not in a hurry, I'll happily play around with this patch next 
week, and both learn a bit and possibly give a better review.

But looking at it right now, this looks great.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55566



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

Reply via email to