[Lldb-commits] [PATCH] D12890: Add plugin.jit-loader.gdb.enable-jit-breakpoint property to make JIT loader breakpoint optional.

[Greg Clayton via lldb-commits]

clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.

Looks good.


================
Comment at: source/Core/PluginManager.cpp:2628
@@ -2627,3 +2627,3 @@
 static lldb::OptionValuePropertiesSP
 GetDebuggerPropertyForPluginsOldStyle (Debugger &debugger,
                                        const ConstString &plugin_type_name,
----------------
ovyalov wrote:
> Do we still need this function? It's used only by platform plugin and there is no platform plugin properties right now in use - we may want to migrate from old style properties.
I believe we do for other branches that we merge into... Leave it for now.


http://reviews.llvm.org/D12890