[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

[Alexander Yermolovich via Phabricator via lldb-commits]

ayermolo added a comment.

In D127889#3604421 <https://reviews.llvm.org/D127889#3604421>, @DavidSpickett wrote:

> I've reverted this due to failures on our bots: https://lab.llvm.org/buildbot/#/builders/96/builds/25032/steps/6/logs/stdio
>
>   /usr/bin/ld: /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb/test/Shell/Commands/Output/dwarf5-low-pc-ranges-inlining.test.tmp.dir/main5.o: Relocations in generic ELF (EM: 62)
>   /usr/bin/ld: /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb/test/Shell/Commands/Output/dwarf5-low-pc-ranges-inlining.test.tmp.dir/main5.o: error adding symbols: file in wrong format
>   collect2: error: ld returned 1 exit status
>
> I think that the system ld is expecting an AArch64 format object here. Perhaps you could use lld instead given that we always build that when running the lldb tests?
>
> Also I'm pretty sure you'll need to require the x86 target in llvm as well. The test will fail if this is an AArch64 target only build for example.

Ah thanks. Will make changes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127889/new/

https://reviews.llvm.org/D127889