[llvm] r182685 - clang-formatted APInt.h

Michael Gottesman mgottesman at apple.com
Fri May 24 16:20:02 PDT 2013


Or actually, that won't help anything. *sigh*.

Ok. I was worried about this. I figured it was ok since I had just fixed up the comments in APInt.h to be consistent. Bad notions. = /.

Michael

On May 24, 2013, at 4:16 PM, Michael Gottesman <mgottesman at apple.com> wrote:

> Ok. Let me do some reverting...
> 
> Michael
> 
> On May 24, 2013, at 4:14 PM, Chris Lattner <clattner at apple.com> wrote:
> 
>> 
>> On May 24, 2013, at 3:38 PM, Michael Gottesman <mgottesman at apple.com> wrote:
>> 
>>> Author: mgottesman
>>> Date: Fri May 24 17:38:49 2013
>>> New Revision: 182685
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=182685&view=rev
>>> Log:
>>> clang-formatted APInt.h
>> 
>> Hi Michael,
>> 
>> Please don't do this.  We don't like widespread formatting changes across the whole sourcebase (e.g. removing trailing whitespace, reformatting, etc) because it is noisy and messes with svn annotate.
>> 
>> It's fine to reformat code that is changing for other reasons, but bulk changes aren't appreciated.
>> 
>> -Chris
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130524/65aeb1da/attachment.html>


More information about the llvm-commits mailing list