[Lldb-commits] [PATCH] D36046: Improve the posix core file triple detection

Tamas Berghammer via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sat Jul 29 06:10:36 PDT 2017

tberghammer added inline comments.

Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:733-735
+  if (target_arch.IsMIPS()) {
     return target_arch;
+  }
Hi Nitesh,

I tried to remove this MIPS specific code as it shouldn't be necessary if I add the above MergeFrom for all architecture but if I do it fails LinuxCore

The issue is that in that case the tripe we get from the core file is "mipsel--" ("mipsel--linux" after the merge) and the one we get from the binary is "mips64el--linux". Is it normal to have a seemingly 32bit core file with a 64bit binary on mips? If not then can I ask you to help me figure out which one is incorrect?

If it is then I don't see any better short term solution then leaving this condition here but it feels like a quite big hack what might backfire when somebody tries to use a core file with a completely incompatible binary.



More information about the lldb-commits mailing list