[llvm-dev] RFC: changing variable naming rules in LLVM codebase
Hubert Tong via llvm-dev
llvm-dev at lists.llvm.org
Thu Feb 14 17:00:05 PST 2019
On Thu, Feb 14, 2019 at 7:32 PM Mehdi AMINI via llvm-dev <
llvm-dev at lists.llvm.org> wrote:
> On Thu, Feb 7, 2019 at 2:11 PM Michael Platings via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>> TL;DR: change the rule for variable names from UpperCamelCase to
>> Just to get wider visibility on this, I'm raising this again as an RFC,
>> as suggested by Roman Lebedev.
>> My original post from last week is here and gives a rationale:
>> There seemed to be general agreement that the status quo is not ideal.
>> Chris Lattner pointed out that this came up in 2014 as well:
>> I've created a patch to implement the change. Review and comments
>> welcome: https://reviews.llvm.org/D57896
> 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.
> (other than that, changing the convention is fine with me, I don't have
> much preference here)
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
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev