<div dir="ltr"><div class="gmail_extra">(since this came up in a few places)<br><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 12:13 AM, Alexei Starovoitov <span dir="ltr"><<a href="mailto:alexei.starovoitov@gmail.com" target="_blank">alexei.starovoitov@gmail.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="">> Variable naming in LLVM is still 'ProudCamelCase', so these should be IsFooBar.<br>
</span>Actually that part of coding style is the most confusing:<br>
$ git grep isVarArg|wc -l<br>
476<br>
$ git grep IsVarArg|wc -l<br>
126<br>
<br>
I see the same in x86, sparc and other backends....<br>
so I think I picked the right one...</blockquote></div><br>So, first, we use humbleCamelCase for function names and ProudCamelCase for variable names, so both will get high counts with this kind of grep.</div><div class="gmail_extra"><br></div><div class="gmail_extra">However, our naming conventions are known to not be consistently enforced across the codebase. We've tried to ask that new code generally follow the new guidelines where doing so doesn't introduce horrible inconsistency. So for a class whose base class's virtual methods all use ProudCamelCase method names, it doesn't make sense to add one new humbleCameCase extra method. But variables almost never have any consistency concerns such as this, and for an entirely new backend such concerns should only really apply to the generic Target interfaces that it is implementing. The rest of the code should be able to follow the coding standards here IMO.</div></div>