clayborg wrote:

> If a module is debug info only, like a dSYM file, and then you try to `target 
> module add` it to a target, it would be nice to add it as the symbol file for 
> an already existing module. The test below, I wouldn't want a target with two 
> things representing a.out, which is what is happening. This can cause lookups 
> to return less valuable results because we will always match `a.out` first, 
> which has no debug info.
> 
> Would it be better to be modify "target symbols add" to allow it to add the 
> symbol file as a module if it returns `true` from a call to 
> "ObjectFile::CanBeTargetModule(...)" but only if there are no matching 
> modules to add the symbol file to?

@JDevlieghere Any comments on the above comment?

https://github.com/llvm/llvm-project/pull/77966
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to