[PATCH] D62105: [CommandLine] Remove OptionCategory and SubCommand caches from the Option class.
Don Hinton via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 3 08:18:52 PDT 2019
hintonda marked an inline comment as done.
hintonda added inline comments.
================
Comment at: llvm/include/llvm/Support/CommandLine.h:1797
+ // Fixes CommandLineTest.AliaswithSubCommand.
+ void addArgument(SubCommand &SC) {}
+
----------------
Will fix this and related methods by making them virtual and adding asserts saying the operation isn't allowed/doesn't make sense in a separate patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62105/new/
https://reviews.llvm.org/D62105
More information about the llvm-commits
mailing list