[Lldb-commits] [lldb] DynamicLoaderDarwin load images in parallel (PR #110439)

[Dmitrii Galimzianov via lldb-commits]

DmT021 wrote:

> it should definitely be enabled by default when we're at the point to merge it, and the setting should only be a safety mechanism if this turns out to cause a problem for a configuration we weren't able to test.

That's fine with me either.

> I would even put it explicitly under an experimental node (e.g. see target.experimental.inject-local-vars we have currently) so if someone does disable it in their ~/.lldbinit file, and we remove the setting in a year or two, they won't get errors starting lldb, it will be silently ignored.

Sure, but can you clarify please should it be named `plugin.dynamic-loader.darwin.experimental.enable-parallel-image-load`  or `plugin.experimental.dynamic-loader.darwin.enable-parallel-image-load` as you wrote previously? Seems like the first option reads as "'enable-parallel-image-load' is an experimental property of 'plugin.experimental.dynamic-loader.darwin'" and the second one as "'dynamic-loader.darwin.enable-parallel-image-load' is an experimental property of 'plugin'".



https://github.com/llvm/llvm-project/pull/110439