[Lldb-commits] [lldb] [LLDB][Minidump] Have Minidumps save off and properly read TLS data (PR #109477)

[Jacob Lalonde via lldb-commits]

Jlalond wrote:

> I think that would make sense, though I also think that the current behavior (attempting to relocate it also does) -- my core question is: how likely is it that the dynamic loader will find the right thread-local for a module which was loaded by the process class, if the two classes can't even agree on its load address?

I'm mentally struggling with this as there is a distinction I think between loading in a core file and actually loading of a module. ProcessMinidump handles loading modules defined in the module list, but the DYLD will need to relocate them. I want to support this workflow but not force it for all Cores (Which I am doing). I wonder if we can force have the process force relocation which is what I'm doing with the `IsCoreFile()` check in DidAttach

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