[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