[Lldb-commits] [PATCH] D12601: Fix TestLoadUnload.test_load_unload for android API > 21

Oleksiy Vyalov via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 3 10:43:44 PDT 2015

ovyalov accepted this revision.
ovyalov added a comment.
This revision is now accepted and ready to land.


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;
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.  

Comment at: source/Target/Platform.cpp:278
@@ +277,3 @@
+                                      if (error.Success() && module_sp)
+                                          module_sp->SetPlatformFileSpec(spec.GetFileSpec());
+                                      return error;
What do you think about putting module_sp->SetPlatformFileSpec(spec.GetFileSpec()) into ModuleList::GetSharedModule ?


More information about the lldb-commits mailing list