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
TestCase.test_mips_n32.
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.
Thanks,
Tamas
https://reviews.llvm.org/D36046
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
