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



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

Reply via email to