[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