<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Oct 5, 2014 at 9:37 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 5 October 2014 07:19, Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br>
> So with that in mind, I would like to ask, would it be possible to consider<br>
> switching to LLVM style for lld?<br>
<br>
</span>We don't usually enforce code styles on side projects because it<br>
doesn't make sense to do so.</blockquote><div><br></div><div>This is not completely accurate. Both LLD and LLDB were given specific exemptions from the coding standards, but Clang wasn't and I wouldn't expect a new subproject to *necessarily* get such an exemption. It might, it might not.</div><div><br></div><div>I consider the current state with both LLD and LLDB a bug rather than a feature because it creates pointless and non-trivial disruption for developers to move between LLVM, Clang, LLD, and LLDB. You can argue that coding standards are like fashion, but *changing* coding standards is a much more pragmatic and real concern. At least one important point of LLVM's standards (in my mind and I suspect other developers' minds) is to drive consistency both within projects and across projects.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Code style changes with time. If the majority of lld developers agree<br>
with you that the current style is bad and needs to be changed (to a<br>
more LLVM-ish style),</blockquote><div><br></div><div>Figuring out whether most LLD devs want to switch seems like the point of the email?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> than future commits should move lld's style<br>
towards that. Not by adding new styles to one line inside an old<br>
function, but by deprecating style when you deprecate code, and<br>
creating style when you create code.<br></blockquote><div><br></div><div>That is one option. But the developers of LLD may be willing to more aggressively convert. We should let them speak for themselves rather than hypothesizing here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
It's also very likely that, by the time you have converted all old<br>
code with new style, the preferred style will have changed, and you'll<br>
want to do it all over again. To avoid that pointless race against<br>
nothing, we tend to keep the style that was in the original<br>
file/function and not mind much.<br></blockquote><div><br></div><div>This has not been my experience in any part of the LLVM project. The coding standards are extremely stable these days.</div></div></div></div>