[Lldb-commits] [PATCH] D17363: Add SymbolFilePDB with basic support for line tables.

[Zachary Turner via lldb-commits]

zturner added a comment.

I looked into this a little bit, it's going to be a couple days of work for me to get termination entries set correctly.  Can I go in as-is?  I know you already Accepted the revision, just want to double check.

I need to go make some changes to `llvm-pdbdump` on the LLVM side (it's just a tool we use for exploratory purposes like `llvm-dwarfdump`) to learn a little bit more about how the API behaves in order to confidently set the termination entry.

So I was thinking I can just make a followup patch to fix that issue.


http://reviews.llvm.org/D17363