[Lldb-commits] [PATCH] D78329: Allow lldb-test to combine -find with -dump-clang-ast

Shafik Yaghmour via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 16 15:38:23 PDT 2020

shafik added inline comments.

Comment at: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:8823
     case clang::Type::Typedef: {
       const clang::TypedefType *typedef_type =
Does it make sense to apply these change to non-objective-C cases?

Comment at: lldb/tools/lldb-test/lldb-test.cpp:726
   if (DumpClangAST) {
-    if (Find != FindType::None)
-      return make_string_error("Cannot both search and dump clang AST.");
-    if (Regex || !Context.empty() || !File.empty() || Line != 0)
-      return make_string_error(
-          "-regex, -context, -name, -file and -line options are not "
-          "applicable for dumping clang AST.");
-    return dumpClangAST;
+    if (Find == FindType::None) {
+      if (Regex || !Context.empty() || !File.empty() || Line != 0)
The relationship between `DumpClangAST` and `Find` is not obvious. Basically we are falling through here in the `FindType::Type` case.

Perhaps a comment here and maybe on the ` Map.Dump(&Stream, false, GetDescriptionLevel());` line might help clarify how these options are related.



More information about the lldb-commits mailing list