[Lldb-commits] [lldb] Allow option to ignore module load errors in ScriptedProcess (PR #127153)

[Med Ismail Bennani via lldb-commits]

medismailben wrote:

> @jimingham @medismailben Can you please let me know if you have more questions regarding the problem and solution ? Right now, I have conditionally ignored the modules which are not created and only loaded the successful ones, if needed I can always load the successfully created modules in scripted process

So we've talked offline with @jimingham and there are 2 things we things should change here:
- ScriptedProcess module loading is too strict (compared to non-ScriptedProcess approach). I don't think having a variable in the ScriptedProcess to skip loading error per module is a good way to address this issue. Instead I'd just make the `ForEach` call not return and log loading errors in the lambda.
> a. Sample Image List of child elf-core target. Note the * at the end, which indicates placeholder object
>
> [0]  0x1234455 [vdso]   /path/to/a.out
> [1] 0x1234566  /path/to/libc.so(*)
> [2]     ...
- Is this behavior something that's specific to ElfCore or is that generic to any process plugin ? If it's the former, I really think this should become an lldb-wide thing. This can be done as a follow-up.

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