[PATCH] D64121: Rename variables so that they start with a lowercase letter.
Chris Lattner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 3 10:19:29 PDT 2019
lattner added a comment.
I'm not a reviewer for LLD, but this change looks really great to me. The major thing to watch out for is that things like "XYZVar" need to be renamed to "xyzVar", but it looks like LLD was already spelling these cases as "XyzVar", so this doesn't come up.
I'm very much +1 on this change, and thing we should roll it out across the entire LLVM project.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64121/new/
https://reviews.llvm.org/D64121
More information about the llvm-commits
mailing list