[Lldb-commits] [PATCH] D98619: [lldb] Use CompileUnit::ResolveSymbolContext in SymbolFileDWARF

Muhammad Omair Javaid via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 22 06:32:51 PDT 2021


omjavaid added a comment.

In D98619#2641136 <https://reviews.llvm.org/D98619#2641136>, @labath wrote:

> In D98619#2641104 <https://reviews.llvm.org/D98619#2641104>, @omjavaid wrote:
>
>> In D98619#2640734 <https://reviews.llvm.org/D98619#2640734>, @labath wrote:
>>
>>> `REQUIRES: x86` disables the test in those builds, which don't have the X86 target enabled (in the build config). That is exactly what we want here -- the test is not actually running this code, so it should succeed regardless of the host architecture. It's working fine on aarch64, for instance (http://lab.llvm.org:8011/#/builders/96/builds/5891). I'm not sure what's the deal with the arm bot (cc @omjavaid). It could be some 32-bit thing. I'm going to try to get a better error message first, and then we'll see what to do next...
>>
>> This is crashing LLDB on arm linux I have reported a bug and marked it xfail. I ll look for a fix later.
>
> Interesting. Crashing actually explains the test output. Thanks for disabling that test. The issue seems to be arm specific as I couldn't reproduce this on i386 lldb (and that makes it pretty weird).
>
> Do you have a backtrace handy by any chance?

I have created http://llvm.org/pr49678. Stack trace attached to pr.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98619



More information about the lldb-commits mailing list