================
@@ -77,12 +78,33 @@ class CommandObjectStatsDump : public CommandObjectParsed {
break;
case 's':
m_stats_options.summary_only = true;
+ // In the "summary" mode, the following sections should be omitted by
+ // default unless their corresponding options are explicitly given.
+ // If such options were processed before 's', `m_seen_options` should
+ // contain them, and we will skip setting them to `false` here. If such
+ // options are not yet processed, we will set them to `false` here, and
+ // they will be overwritten when the options are processed.
+ if (m_seen_options.find((int)'r') == m_seen_options.end())
+ m_stats_options.include_targets = false;
+ if (m_seen_options.find((int)'m') == m_seen_options.end())
+ m_stats_options.include_modules = false;
+ if (m_seen_options.find((int)'t') == m_seen_options.end())
+ m_stats_options.include_transcript = false;
break;
----------------
clayborg wrote:
We should put these smarts into the `StatisticsOptions` class so that we don't
have to do this logic here _and_ this means that when people use the
`SBStatisticsOptions` class via the API, we will do the right thing when using
the API and it will match the behavior here.
https://github.com/llvm/llvm-project/pull/95075
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
