<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 29, 2016 at 5:03 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
> On 2016-Mar-29, at 17:01, Mehdi Amini <<a href="mailto:mehdi.amini@apple.com">mehdi.amini@apple.com</a>> wrote:<br>
><br>
><br>
>> On Mar 29, 2016, at 4:58 PM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br>
>><br>
>><br>
>>> On 2016-Mar-29, at 16:16, Mehdi AMINI <<a href="mailto:mehdi.amini@apple.com">mehdi.amini@apple.com</a>> wrote:<br>
>>><br>
>>> joker.eph updated this revision to Diff 52002.<br>
>>> joker.eph marked 6 inline comments as done.<br>
>>> joker.eph added a comment.<br>
>>><br>
>>> Taking comments into account.<br>
>>><br>
>>><br>
>>> <a href="http://reviews.llvm.org/D18213" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18213</a><br>
>>><br>
>>> Files:<br>
>>> include/llvm/Bitcode/BitstreamReader.h<br>
>>> include/llvm/Bitcode/LLVMBitCodes.h<br>
>>> include/llvm/Bitcode/ReaderWriter.h<br>
>>> include/llvm/IR/ModuleSummaryIndex.h<br>
>>> lib/Bitcode/Reader/BitcodeReader.cpp<br>
>>> lib/Bitcode/Writer/BitcodeWriter.cpp<br>
>>> lib/IR/ModuleSummaryIndex.cpp<br>
>>> lib/Transforms/IPO/FunctionImport.cpp<br>
>>> tools/llvm-as/llvm-as.cpp<br>
>>> tools/llvm-bcanalyzer/llvm-bcanalyzer.cpp<br>
>>><br>
>>> <D18213.52002.patch><br>
>><br>
>> Found a weird spacing issue below (I haven't looked deeply; I figure<br>
>> Rafael and Teresa are already looking at this one?  Happy to look if<br>
>> need be though.)<br>
>><br>
>>> Index: include/llvm/Bitcode/LLVMBitCodes.h<br>
>>> ===================================================================<br>
>>> --- include/llvm/Bitcode/LLVMBitCodes.h<br>
>>> +++ include/llvm/Bitcode/LLVMBitCodes.h<br>
>>> @@ -71,43 +71,46 @@<br>
>>> enum { BITCODE_CURRENT_EPOCH = 0 };<br>
>>><br>
>>>  /// MODULE blocks have a number of optional fields and subblocks.<br>
>>> -  enum ModuleCodes {<br>
>>> -    MODULE_CODE_VERSION     = 1,    // VERSION:     [version#]<br>
>>> -    MODULE_CODE_TRIPLE      = 2,    // TRIPLE:      [strchr x N]<br>
>>> -    MODULE_CODE_DATALAYOUT  = 3,    // DATALAYOUT:  [strchr x N]<br>
>>> -    MODULE_CODE_ASM         = 4,    // ASM:         [strchr x N]<br>
>>> -    MODULE_CODE_SECTIONNAME = 5,    // SECTIONNAME: [strchr x N]<br>
>>> +enum ModuleCodes {<br>
>>> +  MODULE_CODE_VERSION = 1,     // VERSION:     [version#]<br>
>>> +  MODULE_CODE_TRIPLE = 2,      // TRIPLE:      [strchr x N]<br>
>>> +  MODULE_CODE_DATALAYOUT = 3,  // DATALAYOUT:  [strchr x N]<br>
>>> +  MODULE_CODE_ASM = 4,         // ASM:         [strchr x N]<br>
>>> +  MODULE_CODE_SECTIONNAME = 5, // SECTIONNAME: [strchr x N]<br>
>><br>
>> This spacing change seems unrelated (and inconsistent with the enums<br>
>> above and below<br>
><br>
> Oh, that's clang-format, my first iterations on the patch I took care to manually revert the formatting in this file, and then I gave up.<br>
> I can reduce the diff to the only line change on commit though.<br>
<br>
</div></div>If the clang-formatted version is better or not much worse (not always<br>
true for large switch statements and enums), then you can do an NFC<br>
prep commit that clang-formats the entire file and rebase on top of<br>
that.  Saves everyone some hassle.</blockquote><div><br></div><div>I always wondered about that - any time I have touched this file and then run clang-format on my changes it tries to reformat the whole thing and I have to remember to clang-format the individual files in my patch excluding this one. If the clang-format version is ok that would be great to commit it as an NFC change.</div><div><br></div><div>Thanks,</div><div>Teresa </div></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>