[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