<div dir="ltr">Looks like most people in this thread support using LLVM style in LLD. I also had an offline discussion and many people wanted to have one coding style in all LLVM projects. So I'm convinced that we should do that.<div><br></div><div>I'm going to create a patch to rename all variables if no one objects.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 6, 2014 at 3:10 PM, Bruce Hoult <span dir="ltr"><<a href="mailto:bruce@hoult.org" target="_blank">bruce@hoult.org</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"><span class="">On Tue, Oct 7, 2014 at 8:54 AM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>would touch all the lines. Diff is not powerful enough to trace the history beyond variable renaming. svn blame would become useless.</div></div></blockquote><div><br></div></span><div>"useless" is much too strong a description.</div><div><br></div><div>If svn blame points to the reformat commit, then simply re-run the blame on the commit before the reformat.</div><div><br></div><div>If total reformats are frequent then this is impractical. If they happen once in five years then it's not really a big deal. Wrapper scripts can even be pretty easily written to do this automatically.</div><div><br></div></div></div></div>
</blockquote></div><br></div>