[Lldb-commits] [PATCH] D137312: [LLDB] [LoongArch] Add loongarch64 case in ComputeHostArchitectureSupport()

[Tiezhu Yang via Phabricator via lldb-commits]

seehearfeel created this revision.
seehearfeel added reviewers: SixWeining, wangleiat, xen0n, xry111, MaskRay, DavidSpickett.
Herald added a subscriber: StephenFan.
Herald added a project: All.
seehearfeel requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

This is a simple change, loongarch64 host also supports 32-bit binaries,
so note it.

Without this patch:

  [loongson at linux build]$ ./tools/lldb/unittests/Host/HostTests | tail -6
  [==========] 78 tests from 18 test suites ran. (16 ms total)
  [  PASSED  ] 77 tests.
  [  FAILED  ] 1 test, listed below:
  [  FAILED  ] HostTest.GetProcessInfo
  
   1 FAILED TEST

With this patch:

  [loongson at linux build]$ ./tools/lldb/unittests/Host/HostTests | tail -2
  [==========] 78 tests from 18 test suites ran. (15 ms total)
  [  PASSED  ] 78 tests.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D137312

Files:
  lldb/source/Host/common/HostInfoBase.cpp


Index: lldb/source/Host/common/HostInfoBase.cpp
===================================================================
--- lldb/source/Host/common/HostInfoBase.cpp
+++ lldb/source/Host/common/HostInfoBase.cpp
@@ -340,6 +340,7 @@
   case llvm::Triple::ppc64le:
   case llvm::Triple::x86_64:
   case llvm::Triple::riscv64:
+  case llvm::Triple::loongarch64:
     arch_64.SetTriple(triple);
     arch_32.SetTriple(triple.get32BitArchVariant());
     break;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137312.472832.patch
Type: text/x-patch
Size: 451 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20221103/f1658e0e/attachment.bin>