johanengelen created this revision.
johanengelen added a reviewer: LLDB.
johanengelen added a subscriber: lldb-commits.

Binaries with Dwarf language set to "DW_LANG_D" currently do not show much 
information in LLDB (e.g. no local variables).

With this simple change, Clang language support is used for D (the same as what 
is done for Rust) and local variables show up nicely.

I don't know whether, or how, to add a testcase for this. Please advise.

(I don't have write access, so if you accept the change, please apply it too. 
Thanks!)

https://reviews.llvm.org/D24794

Files:
  source/Symbol/ClangASTContext.cpp

Index: source/Symbol/ClangASTContext.cpp
===================================================================
--- source/Symbol/ClangASTContext.cpp
+++ source/Symbol/ClangASTContext.cpp
@@ -113,7 +113,9 @@
          Language::LanguageIsPascal(language) ||
          // Use Clang for Rust until there is a proper language plugin for it
          language == eLanguageTypeRust ||
-         language == eLanguageTypeExtRenderScript;
+         language == eLanguageTypeExtRenderScript ||
+         // Use Clang for D until there is a proper language plugin for it
+         language == eLanguageTypeD;
 }
 }
 


Index: source/Symbol/ClangASTContext.cpp
===================================================================
--- source/Symbol/ClangASTContext.cpp
+++ source/Symbol/ClangASTContext.cpp
@@ -113,7 +113,9 @@
          Language::LanguageIsPascal(language) ||
          // Use Clang for Rust until there is a proper language plugin for it
          language == eLanguageTypeRust ||
-         language == eLanguageTypeExtRenderScript;
+         language == eLanguageTypeExtRenderScript ||
+         // Use Clang for D until there is a proper language plugin for it
+         language == eLanguageTypeD;
 }
 }
 
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to