EricWF added a comment. In http://reviews.llvm.org/D16948#345722, @dexonsmith wrote:
> Light review below, looks like great work. > > I noticed the tests seem somewhat inconsistent about whether to > glue `&` to the type or use two spaces (i.e., ` & `). Which one is > preferred in this codebase? Or is it laissez-faire? I prefer it glued to the type *i think* but there is no set style currently. I'll try and make it more consistent as I see them. > The rest of my comments are inline. Thanks! If I could bother you to put them on phab next time instead of email it really helps me track them. http://reviews.llvm.org/D16948 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
