[PATCH] D83979: [clang][cli] Port LangOpts option flags to new option parsing system
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 16 19:52:09 PST 2020
MaskRay added inline comments.
================
Comment at: clang/include/clang/Driver/Options.td:1001
+ "LangOpts->AccessControl", DefaultsToTrue,
+ ChangedBy<NegFlag, [], "Disable C++ access control">,
+ ResetBy<PosFlag>>;
----------------
`ChangedBy<NegFlag, ` and `ResetBy<PosFlag>` are a bit verbose. How about a shorthand for this most common form?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83979/new/
https://reviews.llvm.org/D83979
More information about the cfe-commits
mailing list