[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