aaron.ballman added a comment.

For background: what is Fuchsia and where do these requirements come from (are 
they documented publicly somewhere)?

We tend to prefer concise patches over code dumps, so I think it would make 
sense to split this review into multiple patches. The first one can be adding 
the module and a single check along with background information on why the 
module would be appropriate. The rest of the patches should be one check per 
patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D40108



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

Reply via email to