<div dir="ltr">By "in style", I meant style in general and did not mean the documented coding style. Code is easy to read and understand if it is written consistently and in the same taste throughout its code base, so I want new code looks and feels similar to the existing code. Following the documented coding style is the minimum standard to achieve it, but it's not everything. I think that's what we were talking about.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 6, 2016 at 2:37 PM, Davide Italiano <span dir="ltr"><<a href="mailto:dccitaliano@gmail.com" target="_blank">dccitaliano@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davide added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D22015#475966" rel="noreferrer" target="_blank">http://reviews.llvm.org/D22015#475966</a>, @ruiu wrote:<br>
<br>
> LGTM<br>
><br>
> OK, I'll make a follow-up patch for this, but I'd like to say that it is important to keep code consistent in style and taste, and code review is a way to achieve it, so I'm not nitpicking.<br>
<br>
<br>
<br>
<br>
</span><span class="">In <a href="http://reviews.llvm.org/D22015#475966" rel="noreferrer" target="_blank">http://reviews.llvm.org/D22015#475966</a>, @ruiu wrote:<br>
<br>
> LGTM<br>
><br>
> OK, I'll make a follow-up patch for this, but I'd like to say that it is important to keep code consistent in style and taste, and code review is a way to achieve it, so I'm not nitpicking.<br>
<br>
<br>
</span>For this reason some projects have an official style guide, e.g. <a href="https://www.freebsd.org/cgi/man.cgi?query=style&sektion=9" rel="noreferrer" target="_blank">https://www.freebsd.org/cgi/man.cgi?query=style&sektion=9</a><br>
<br>
Also, please note that LLVM is not always consistent with itself, and lld is a clear example.<br>
Two differences that come to my mind are:<br>
<br>
1. Lto instead of LTO, Elf instead of ELF<br>
2. Not using auto unless it's really obvious from the context<br>
<br>
So having a full set of "rules" or recommendations somewhere will be ideal, and I hope you can contribute those as it seem you care about the most. We have codingstandards but it doesn't seem to cover everything you want.<br>
<br>
Thanks!<br>
<br>
<br>
<a href="http://reviews.llvm.org/D22015" rel="noreferrer" target="_blank">http://reviews.llvm.org/D22015</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>