<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 14, 2019 at 7:32 PM Mehdi AMINI via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 7, 2019 at 2:11 PM Michael Platings via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">




<div dir="ltr">
<div id="gmail-m_-3544027406910407196gmail-m_1204835170396394775divtagdefaultwrapper" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Helvetica,sans-serif" dir="ltr">
<p style="margin-top:0px;margin-bottom:0px">TL;DR: change the rule for variable names from UpperCamelCase to lowerCamelCase.<br>
</p>
<p style="margin-top:0px;margin-bottom:0px"><br>
</p>
<p style="margin-top:0px;margin-bottom:0px">Just to get wider visibility on this, I'm raising this again as an RFC, as suggested by
<span>Roman Lebedev</span>.</p>
<p style="margin-top:0px;margin-bottom:0px"><br>
</p>
<p style="margin-top:0px;margin-bottom:0px">My original post from last week is here and gives a rationale: <a href="http://lists.llvm.org/pipermail/llvm-dev/2019-February/129854.html" class="gmail-m_-3544027406910407196gmail-m_1204835170396394775OWAAutoLink" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2019-February/129854.html</a>.
 There seemed to be general agreement that the status quo is not ideal.<br>
</p>
<p style="margin-top:0px;margin-bottom:0px"><br>
</p>
<p style="margin-top:0px;margin-bottom:0px">Chris Lattner pointed out that this came up in 2014 as well: <a href="http://lists.llvm.org/pipermail/llvm-dev/2014-October/077685.html" class="gmail-m_-3544027406910407196gmail-m_1204835170396394775OWAAutoLink" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2014-October/077685.html</a><br>
</p>
<p style="margin-top:0px;margin-bottom:0px"><br>
</p>
<p style="margin-top:0px;margin-bottom:0px">I've created a patch to implement the change. Review and comments welcome:
<a href="https://reviews.llvm.org/D57896" class="gmail-m_-3544027406910407196gmail-m_1204835170396394775OWAAutoLink" target="_blank">https://reviews.llvm.org/D57896</a></p></div></div></blockquote><div><br></div><div><br></div><div>I'm concerned about the internal inconsistency we would live with for (likely) many years if there is not a migration plan to converge the code base toward the new naming convention.</div><div>(other than that, changing the convention is fine with me, I don't have much preference here)</div></div></div></blockquote><div>I think consistency within a function (for locals) and within a class (for class members) is enough. I don't think that is any worse than the consistency over</div><div>int *p;</div><div>versus</div><div>int* p;</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>-- </div><div>Mehdi</div><div> </div></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>