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

Reply via email to