[PATCH] D43313: [DebugInfo] Support parsing DWARF expressions

[Rafael Auler via Phabricator via llvm-commits]

rafauler added a comment.

Thanks Adrian, I'll drop the \brief. That makes sense, thanks for the context on DataExtractor versus other approaches. In our client code, it never made a lot of sense to use DataExtractor and that's part of the reason I was lifting DWARF decoding helpers to live on their own. Then I was using readobj as a proxy for our client code so we have something upstream that demonstrates how we are accessing and manipulating DWARF CFIs (both coming from FDEs and CIEs).  Then I'll take a second look on this design to avoid pitfalls with unchecked bounds.


Repository:
  rL LLVM

https://reviews.llvm.org/D43313