[PATCH] D105881: [flang][driver] Switch to `BoolFOption` for boolean options
Andrzej Warzynski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 13 02:52:07 PDT 2021
awarzynski added inline comments.
================
Comment at: clang/include/clang/Driver/Options.td:246
+class EmptyKPM<string base> : KeyPathAndMacro<"", "", ""> {}
class DiagnosticOpts<string base>
----------------
@jansvoboda11 , is this the right approach here? I'd like use `BoolFOption` in Flang, but we don't need `KeyPathAndMacro` just yet. We may do in the future.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105881/new/
https://reviews.llvm.org/D105881
More information about the cfe-commits
mailing list