zaks.anna added a comment.

Could you please split the patch into two - one with the previously reviewed 
support for functions that take a single size value and another patch that 
models the two size arguments (num and size). It's easier to review patches if 
they do not grow new functionality. Splitting the patch would also play nicely 
with the incremental development policy of LLVM.

Thanks!


Repository:
  rL LLVM

https://reviews.llvm.org/D28348



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

Reply via email to