Re: [Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-12-01 Thread Ewan Crawford via lldb-commits
EwanCrawford added a comment. Thanks for fixing this Adrian Repository: rL LLVM http://reviews.llvm.org/D15092 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-11-30 Thread Adrian McCarthy via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL254338: Fix hang in global static initialization (authored by amccarth). Changed prior to commit: http://reviews.llvm.org/D15092?vs=41433&id=41435#toc Repository: rL LLVM http://reviews.llvm.org/D15

Re: [Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-11-30 Thread Zachary Turner via lldb-commits
zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land. Ahh ok if there's already precedent for returning by reference then it's fine. http://reviews.llvm.org/D15092 ___ lldb-commits mailing list lld

Re: [Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-11-30 Thread Adrian McCarthy via lldb-commits
amccarth added inline comments. Comment at: source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp:185 @@ -184,3 +184,3 @@ -static const ConstString FallbackStructName; // Print this as the type name of a struct Element

Re: [Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-11-30 Thread Zachary Turner via lldb-commits
zturner added inline comments. Comment at: source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp:185 @@ -184,3 +184,3 @@ -static const ConstString FallbackStructName; // Print this as the type name of a struct Element

[Lldb-commits] [PATCH] D15092: Fix hang in global static initialization

2015-11-30 Thread Adrian McCarthy via lldb-commits
amccarth created this revision. amccarth added reviewers: EwanCrawford, zturner. amccarth added a subscriber: lldb-commits. A previous patch introduced a global static ConstString instance. On Windows, this results in a hang because constructing the string pool appears to depend on other global