[Lldb-commits] [PATCH] D61853: [FuncUnwinders] Use "symbol file" unwind plans for unwinding

[Pavel Labath via Phabricator via lldb-commits]

labath updated this revision to Diff 200737.
labath added a comment.

- use (reduced) yaml form of the minidump now that yaml2obj supports memory regions
- move the symbol file unwind plan to the top of the list. I agree with the reasoning behind this and with the debug_frame>eh_frame (which I'll create a separate patch for) idea, though I think this is mostly academic, as we're unlikely to ever have two of these plans describing the same function.


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

https://reviews.llvm.org/D61853

Files:
  include/lldb/Symbol/FuncUnwinders.h
  lit/SymbolFile/Breakpad/Inputs/unwind-via-stack-cfi.syms
  lit/SymbolFile/Breakpad/Inputs/unwind-via-stack-cfi.yaml
  lit/SymbolFile/Breakpad/stack-cfi-parsing.test
  lit/SymbolFile/Breakpad/unwind-via-stack-cfi.test
  source/Commands/CommandObjectTarget.cpp
  source/Plugins/Process/Utility/RegisterContextLLDB.cpp
  source/Symbol/FuncUnwinders.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61853.200737.patch
Type: text/x-patch
Size: 12076 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190522/8f8f7bee/attachment-0001.bin>