tberghammer added inline comments.
================
Comment at: source/Core/Module.cpp:1712
@@ -1712,1 +1711,3 @@
+ if (!FileSpec::Equal (file_spec, m_file,
(bool)file_spec.GetDirectory()) &&
+ !FileSpec::Equal (file_spec, m_platform_file,
(bool)file_spec.GetDirectory()))
return false;
----------------
ovyalov wrote:
> It might expose some tricky problems when host and target have the same
> library with the same path but different content.. - please make sure that
> there is no regressions.
I run the test suit and seen no regression.
If we have a library with the same path on the host and on the target and we
don't have UUID then we are already having issues differentiating between them.
In the case you described we treat them as matching modules even before my
change.
================
Comment at: source/Target/Platform.cpp:278
@@ +277,3 @@
+ if (error.Success() && module_sp)
+
module_sp->SetPlatformFileSpec(spec.GetFileSpec());
+ return error;
----------------
ovyalov wrote:
> What do you think about putting
> module_sp->SetPlatformFileSpec(spec.GetFileSpec()) into
> ModuleList::GetSharedModule ?
I would like to limit the scope of this change to the case when remote
debugging is used. It can be confusing if we set platform file spec even when
no remote platform is involved
http://reviews.llvm.org/D12601
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
