[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

Mon Jun 12 09:36:23 PDT 2023

Michael137 added a comment.

In D144999#4409671 <https://reviews.llvm.org/D144999#4409671>, @oontvoo wrote:

> P.S This might have the same root-cause with the previous comment and could be fixed by D152540 <https://reviews.llvm.org/D152540> as well.
> (not able to repro the failures yet - don't have arm64 readily available, will test it later this evening. but on linux x86-64 and macos x86-64, ninja check-debuginfo passed for me)

FYI, still failing with D152540 <https://reviews.llvm.org/D152540>. I'm able to repro this on my arm64 machine.

The repro without running dexter is as follows:

  ./bin/clang++ cross-project-tests/debuginfo-tests/dexter-tests/optnone-struct-and-methods.cpp -g -O0 -o test.out
  ./bin/lldb test.out -o "br se -l 87" -o run -o s -o s -o s -o s -o "expr _data.b.other_b"
  (lldb) expr _data.b.other_b 
  (A::B) $0 = 0x10d50         

We would however expect the value to be `B_VALUE`

The backtrace also looks wonky:

  (lldb) bt
  * thread #1, queue = 'com.apple.main-thread', stop reason = step in
    * frame #0: 0x0000000100003d18 test.out`(anonymous namespace)::A::getData(this=0x0000000100011910) at optnone-struct-and-methods.cpp:87:5
      frame #1: 0x0000000187185058 dyld`start + 2224

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144999