<br><br><div class="gmail_quote">On Wed Oct 08 2014 at 7:25:29 PM Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 8, 2014 at 10:23 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"></div></div></div></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div>On Wed, Oct 8, 2014 at 10:16 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><span><br><div class="gmail_quote">On Wed, Oct 8, 2014 at 9:34 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>> On Oct 8, 2014, at 1:55 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
><br>
> On 8 October 2014 05:25, Chris Lattner <<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>> wrote:<br>
>>> Up until now the thread has been about “formatting”.  You suggested renaming<br>
>>> every variable in the project!<br>
>><br>
>> If that's what it takes.<br>
><br>
> If we're still talking about applying it to the HEAD, not the whole<br>
> history, I think it's feasible.<br>
<br>
</span>Yep, to be clear, I mean one big change to head, not trying to rewrite history.</blockquote></div><br></span>I think rewriting history is kind of a hand-wavy pipe dream. ;] I'll believe it when I see it. For now, we should stick to what we know we could actually do, and changing head is the only realistic option there.</div></div></blockquote></div></div></div></div></div></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br>FWIW - I don't think we even have that power (when it comes to global renaming) yet, do we? (rename everything in a project to match some naming convention)</div></div></div></div></blockquote><div><br></div><div>Behold the power of perl.</div><div><br></div><div>More realistically, I think Manuel has cooked up some one-off tools specifically for renaming variables to match LLVM's conventions. Adding him.</div></div></div></div></blockquote><div><br></div><div>That tool was an early one-off that I used to port some code, but it was just enough to port that code. It shouldn't be too hard to code something up that works for lld (it would need to use clang-modernize's 2-phase code transformation stuff). </div><div>cc'ing Alex & Benjamin, who afaik have thought a bit about how to get this stuff to work with clang-tidy.</div><div><br></div><div>On a different note:</div><div>My favorite feature of Phabricator: if you look at a file in blame mode, there are little "<<" next to the revision, which lets you jump to the file "past that revision". This lets one quickly browse to the right point for what one is looking for, making naming or formatting changes less burdensome.</div><div>Example: <a href="http://reviews.llvm.org/diffusion/L/browse/cfe/trunk/include/clang/Basic/DiagnosticIDs.h;198376">http://reviews.llvm.org/diffusion/L/browse/cfe/trunk/include/clang/Basic/DiagnosticIDs.h;198376</a></div><div><br></div><div>Cheers,</div><div>/Manuel</div></div>