[PATCH] D129129: CommandLine: Cleanup options and remove use of ManagedStatic

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/include/llvm/Support/CommandLine.h:354
   /// This option must have been the last option registered.
   /// For testing purposes only.
   void removeArgument();
----------------
"This option must have been the last option registered. For testing purposes only."

Is this still accurate?  Is it an issue here?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129129/new/

https://reviews.llvm.org/D129129