labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
lgtm ================ Comment at: source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp:1128 - RenderScriptRuntime *lang_rt = - (RenderScriptRuntime *)exe_ctx.GetProcessPtr()->GetLanguageRuntime( - eLanguageTypeExtRenderScript); + RenderScriptRuntime *lang_rt = llvm::cast_or_null<RenderScriptRuntime>( + exe_ctx.GetProcessPtr()->GetLanguageRuntime( ---------------- nit: the code (in this whole file) is clearly assuming the pointer is non-null, so you don't need the _or_null part. :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62934/new/ https://reviews.llvm.org/D62934 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
