[PATCH] D57896: Variable names rule
Chris Lattner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 19 05:04:28 PST 2019
lattner added a comment.
> Changed recommendation for acronyms from lower case to upper case, as suggested by several responses to the RFC.
I haven't been following the discussion closely - why is this the preferred direction? I don't think that things like "Basicblock *bb" or "MachineInstr *mi" will be confusing, and going towards a consistently leading lower case letter seems simple and preferable.
-Chris
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57896/new/
https://reviews.llvm.org/D57896
More information about the cfe-commits
mailing list