Skip to content

Commit

Permalink
Reland "[LLDB] [LoongArch] Add loongarch64 case in ComputeHostArchite…
Browse files Browse the repository at this point in the history
…ctureSupport()"

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

Without this patch:

```
[loongson@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@linux build]$ ./tools/lldb/unittests/Host/HostTests | tail -2
[==========] 78 tests from 18 test suites ran. (15 ms total)
[  PASSED  ] 78 tests.
```

Reviewed By: SixWeining, xen0n, MaskRay, DavidSpickett

Differential Revision: https://reviews.llvm.org/D137312
  • Loading branch information
Tiezhu Yang authored and SixWeining committed Nov 4, 2022
1 parent b3578f3 commit 8cca7f3
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lldb/source/Host/common/HostInfoBase.cpp
Expand Up @@ -340,6 +340,7 @@ void HostInfoBase::ComputeHostArchitectureSupport(ArchSpec &arch_32,
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;
Expand Down

0 comments on commit 8cca7f3

Please sign in to comment.