[PATCH] D93497: Salvage debug info for function arguments in coro-split funclets.

Tue Jan 12 19:57:17 PST 2021

dongAxis1944 added a comment.

I do not see any other problems, but if the code use c++ program, it will cause lldb read invalid address like I write before:

  Process 106638 stopped
  * thread #1, name = 'a.out', stop reason = breakpoint 1.2
      frame #0: 0x0000000000401bd0 a.out`foo(t=0x0000000000000000) at coro-debug-1.cpp:39:26
     36     //   (int) i = 1
     37
     38     co_await suspend_always();
  -> 39     printf("%d, %d, %d\n", i, j, t.i); // 2, 1
                                   ^
     40     // Breakpoint 2:
     41     //   (lldb) frame variable i
     42     //   (int) i = <variable not available>
  (lldb) p t
  error: Couldn't apply expression side effects : Couldn't dematerialize a result variable: couldn't read its memory: 0     ---->

In D93497#2494295 <https://reviews.llvm.org/D93497#2494295>, @aprantl wrote:

> It sounds like the bug @dongAxis1944 has found is unrelated to this patch. Should we move forward with this one, or are there any other problems you see?

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

https://reviews.llvm.org/D93497