andreybokhanko added a comment. In http://reviews.llvm.org/D11297#235525, @rjmccall wrote:
> Yes, please make it an error. Done. John, thank you for all your patience and explanations! -- I understand that this particular review and patch author required more than the usual measure. :-( > And the obvious test changes are fine. :) I asked because after switching from warning to error, I had to introduce a new run line in the test -- effectively transforming it into two. Andrey Repository: rL LLVM http://reviews.llvm.org/D11297 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits