From 1e05c3bc28258afdbf3a9d7c4ba24e2dbc00034c Mon Sep 17 00:00:00 2001 From: Greg Clayton Date: Thu, 24 Mar 2016 21:48:10 +0000 Subject: Get rid of two global constructors by making things static variables in the only function that uses these variables. llvm-svn: 264347 --- .../RenderScriptRuntime/RenderScriptRuntime.cpp | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp') diff --git a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp index 654a8acec62..8d1dfdcf4f7 100644 --- a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp +++ b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp @@ -757,8 +757,6 @@ const uint32_t RenderScriptRuntime::AllocationDetails::RSTypeToFormat[][3] = { {eFormatVectorOfFloat32, eFormatVectorOfFloat32, sizeof(float) * 4} // RS_TYPE_MATRIX_2X2 }; -const std::string RenderScriptRuntime::s_runtimeExpandSuffix(".expand"); -const std::array RenderScriptRuntime::s_runtimeCoordVars{{"rsIndex", "p->current.y", "p->current.z"}}; //------------------------------------------------------------------ // Static Functions //------------------------------------------------------------------ @@ -3267,6 +3265,9 @@ RenderScriptRuntime::GetFrameVarAsUnsigned(const StackFrameSP frame_sp, const ch bool RenderScriptRuntime::GetKernelCoordinate(RSCoordinate &coord, Thread *thread_ptr) { + static const std::string s_runtimeExpandSuffix(".expand"); + static const std::array s_runtimeCoordVars{{"rsIndex", "p->current.y", "p->current.z"}}; + Log *log(GetLogIfAnyCategoriesSet(LIBLLDB_LOG_LANGUAGE)); if (!thread_ptr) @@ -3299,13 +3300,13 @@ RenderScriptRuntime::GetKernelCoordinate(RSCoordinate &coord, Thread *thread_ptr // Check if function name has .expand suffix std::string func_name(func_name_cstr); - const int length_difference = func_name.length() - RenderScriptRuntime::s_runtimeExpandSuffix.length(); + const int length_difference = func_name.length() - s_runtimeExpandSuffix.length(); if (length_difference <= 0) continue; const int32_t has_expand_suffix = func_name.compare(length_difference, - RenderScriptRuntime::s_runtimeExpandSuffix.length(), - RenderScriptRuntime::s_runtimeExpandSuffix); + s_runtimeExpandSuffix.length(), + s_runtimeExpandSuffix); if (has_expand_suffix != 0) continue; @@ -3315,12 +3316,12 @@ RenderScriptRuntime::GetKernelCoordinate(RSCoordinate &coord, Thread *thread_ptr // Get values for variables in .expand frame that tell us the current kernel invocation bool found_coord_variables = true; - assert(RenderScriptRuntime::s_runtimeCoordVars.size() == coord.size()); + assert(s_runtimeCoordVars.size() == coord.size()); for (uint32_t i = 0; i < coord.size(); ++i) { uint64_t value = 0; - if (!GetFrameVarAsUnsigned(frame_sp, RenderScriptRuntime::s_runtimeCoordVars[i], value)) + if (!GetFrameVarAsUnsigned(frame_sp, s_runtimeCoordVars[i], value)) { found_coord_variables = false; break; -- cgit v1.2.3