[PATCH] D142932: [NFC] Multilib YAML parsing
Petr Hosek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 14 01:35:54 PST 2023
phosek added a comment.
For the existing YAML files within LLVM, we most commonly used the `PascalCase` for field names and less commonly `lisp-case`. This change uses `camelCase` which is unusual and unless there's a particular reason for using this scheme such as compatibility with an existing format, I'd prefer keeping the format consistent with the rest of LLVM and using `PascalCase` or `lisp-case`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142932/new/
https://reviews.llvm.org/D142932
More information about the cfe-commits
mailing list