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

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 1 12:03:01 PST 2016


clayborg added a comment.

See inlined comments.


================
Comment at: source/Target/Language.cpp:272
@@ -271,2 +271,3 @@
         case eLanguageTypeC11:
+        case eLanguageTypeExtRenderScript:
             return true;
----------------
Why is this needed? Seems like this is incorrect. Are you trying to ask if a language uses the clang::ASTContext? Please explain why renderscript is C.


Repository:
  rL LLVM

http://reviews.llvm.org/D16766





More information about the lldb-commits mailing list