labath added a comment.

Try this for patch for size: F10586804: a.patch 
<https://reviews.llvm.org/F10586804>. It needs to me moved into a separate test 
case, and cleaned up a bit, but I believe it should trigger the bug you're 
fixing here.

The tricky part about unwind info is that you currently need a Process instance 
around even to just inspect (= parse) it. I have ideas how to fix that, but no 
time to implement them :(. Until then we need to create a process somehow to 
test unwind info parsing.

Fortunately, these days we have a minidump files and their yaml representation, 
which gives us a relatively easy (though there definitely are ways to improve 
that too) method to create Processes from core files. I consider something like 
this to be the future of testing unwinding, because it's usually very hard to 
reliably drive a real process to a point which will exercise some specific 
unwind corner case. OTOH, with a minidump(core) file, you can just freeze a 
process in the state that you're intersted in, and the test will always test 
the exact same thing.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D69502



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to