[PATCH] D71228: [Commandline] Move Debug options from Debug.cpp to Commandline.cpp

[serge via Phabricator via llvm-commits]

serge-sans-paille added a comment.

Thanks @hintonda for the clarification! I've added an extra question, LGTM otherwise.



================
Comment at: llvm/lib/Support/Debug.cpp:82
 
-// -debug - Command line option to enable the DEBUG statements in the passes.
-// This flag may only be enabled in debug builds.
-static cl::opt<bool, true>
-Debug("debug", cl::desc("Enable debug output"), cl::Hidden,
-      cl::location(DebugFlag));
-
-// -debug-buffer-size - Buffer the last N characters of debug output
-//until program termination.
-static cl::opt<unsigned>
-DebugBufferSize("debug-buffer-size",
-                cl::desc("Buffer the last N characters of debug output "
-                         "until program termination. "
-                         "[default 0 -- immediate print-out]"),
-                cl::Hidden,
-                cl::init(0));
-
-static cl::list<std::string, ManagedStatic<std::vector<std::string>>>
-    DebugOnly("debug-only",
-              cl::desc("Enable a specific type of debug output (comma "
-                       "separated list of types)"),
-              cl::Hidden, cl::ZeroOrMore, cl::value_desc("debug string"),
-              cl::location(CurrentDebugType), cl::CommaSeparated,
-              cl::callback([](const std::string &) { DebugFlag = true; }));
+unsigned DebugBufferSize = 0;
 
----------------
Why did you keep that definition here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71228