[Lldb-commits] [clang] [lldb] [lldb] Add filter option to AST dump command (PR #142164)

[Michael Buch via lldb-commits]

================
@@ -2235,11 +2235,22 @@ class CommandObjectTargetModulesDumpClangAST
       : CommandObjectTargetModulesModuleAutoComplete(
             interpreter, "target modules dump ast",
             "Dump the clang ast for a given module's symbol file.",
-            //"target modules dump ast [<file1> ...]")
-            nullptr, eCommandRequiresTarget) {}
+            "target modules dump ast [--filter <name>] [<file1> ...]",
+            eCommandRequiresTarget),
+        m_filter(LLDB_OPT_SET_1, false, "filter", 'f', 0, eArgTypeName, "TODO",
+                 /*default_value=*/"") {
+    m_option_group.Append(&m_filter, LLDB_OPT_SET_ALL, LLDB_OPT_SET_1);
+    m_option_group.Finalize();
+    AddSimpleArgumentList(eArgTypeFilename, eArgRepeatStar);
----------------
Michael137 wrote:

This is also probably unnecessary 

https://github.com/llvm/llvm-project/pull/142164