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

Ewan Crawford via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 1 03:50:12 PST 2016


EwanCrawford created this revision.
EwanCrawford added a reviewer: clayborg.
EwanCrawford added a subscriber: lldb-commits.
EwanCrawford set the repository for this revision to rL LLVM.

A DWARF language vender extension for RenderScript was added to LLVM in r259348(http://reviews.llvm.org/D16409)
We should use this generated enum instead of the hardcoded value.

RenderScript is also C99 with some extensions, so when it's detected we should treat it as C. 

Repository:
  rL LLVM

http://reviews.llvm.org/D16766

Files:
  source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
  source/Target/Language.cpp

Index: source/Target/Language.cpp
===================================================================
--- source/Target/Language.cpp
+++ source/Target/Language.cpp
@@ -269,6 +269,7 @@
         case eLanguageTypeC89:
         case eLanguageTypeC99:
         case eLanguageTypeC11:
+        case eLanguageTypeExtRenderScript:
             return true;
         default:
             return false;
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.46520.patch
Type: text/x-patch
Size: 945 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160201/2953dfbd/attachment.bin>


More information about the lldb-commits mailing list