[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