[Lldb-commits] [PATCH] D16766: [RenderScript] Use LLVM DWARF language enum

[Ewan Crawford via lldb-commits]

EwanCrawford updated this revision to Diff 46656.
EwanCrawford added a comment.

Thanks for taking a look, yes I want to use ClangASTContext.
Although RenderScript is mostly just C99 checking for it in a function called `LanguageIsC` does seem off.

I've put the comparison directly in ClangASTContextSupportsLanguage(), but let me know if you'd prefer a more generic solution for allowing new languages to use ClangASTContext.


Repository:
  rL LLVM

http://reviews.llvm.org/D16766

Files:
  source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
  source/Symbol/ClangASTContext.cpp

Index: source/Symbol/ClangASTContext.cpp
===================================================================
--- source/Symbol/ClangASTContext.cpp
+++ source/Symbol/ClangASTContext.cpp
@@ -105,7 +105,8 @@
         return language == eLanguageTypeUnknown || // Clang is the default type system
                Language::LanguageIsC (language) ||
                Language::LanguageIsCPlusPlus (language) ||
-               Language::LanguageIsObjC (language);
+               Language::LanguageIsObjC (language) ||
+               language == eLanguageTypeExtRenderScript;
     }
 }
 
Index: source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
===================================================================
--- source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
+++ source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
@@ -1180,7 +1180,7 @@
     {
     case DW_LANG_Mips_Assembler:
         return eLanguageTypeMipsAssembler;
-    case 0x8e57: // FIXME: needs to be added to llvm
+    case DW_LANG_GOOGLE_RenderScript:
         return eLanguageTypeExtRenderScript;
     default:
         return static_cast<LanguageType>(val);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16766.46656.patch
Type: text/x-patch
Size: 1133 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160202/9ea5d5e8/attachment-0001.bin>