<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 25, 2015 at 11:52 PM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>><br>
>> Author: grimar<br>
>> Date: Wed Nov 25 16:15:01 2015<br>
>> New Revision: 254105<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=254105&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=254105&view=rev</a><br>
>> Log:<br>
>> Reapply r254098.<br>
><br>
>It's helpful to include the original commit message when recommitting a patch so it stands alone/is informative.<br>
</span>Sorry, I focused on fix part and missed that. Next time will do.<br>
<span class=""><br>
>><br>
>> Fix is (OutputSections.cpp):<br>
>> for (std::pair<const SymbolBody *, size_t> &I : Entries) {<br>
>>  =><br>
>> for (std::pair<const SymbolBody *, unsigned> &I : Entries) {<br>
><br>
>It might be best to use 'auto' to avoid this bug<br>
<br>
</span>Yes, but that one passed the review and I didnt wanted to touch it a lot because of that. Was not sure about auto here, tried to make minimal changes just to fix the buildbot.<br></blockquote><div><br></div><div>If it helps, I'll make it a post-commit review request: please change this to auto because of the subtleties here.<br><br>Was this caught by a buildbot with Clang's warning, or due to a test failing on a machine where size_t != unsigned?<br><br>- Dave</div><div> </div></div><br></div></div>