Hi llvm-dev, I’ve put together an implementation of a set of C API bindings for the DIBuilder interface. This builds off of work the Rust team did to wrap the DebugInfo API for rustc. It’s been expanded from their initial implementation and slightly generalized. I’d like to get some feedback before I submit the patch to Phabricator. Specifically, I’d like to know what I should add to it and what I should potentially leave out. I’d also like to know if this has a good chance of being accepted, or if there’s a reason I didn’t know why it wouldn’t. The current state of changes are visible here: https://github.com/llvm-mirror/llvm/compare/master...harlanhaskins:debug-info-api Thanks! Harlan Haskins @harlanhaskins
Hi, Please make sure to add Eric Christopher as a reviewer. I suspect the main challenge is that the C API is supposed to be stable, and if I understand correctly, debug info representation have historically not been very stable. But I’ll leave this to the experts... — Mehdi> On Apr 16, 2017, at 4:32 PM, Harlan Haskins via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > Hi llvm-dev, > > I’ve put together an implementation of a set of C API bindings for the DIBuilder interface. This builds off of work the Rust team did to wrap the DebugInfo API for rustc. It’s been expanded from their initial implementation and slightly generalized. > > I’d like to get some feedback before I submit the patch to Phabricator. Specifically, I’d like to know what I should add to it and what I should potentially leave out. I’d also like to know if this has a good chance of being accepted, or if there’s a reason I didn’t know why it wouldn’t. > > The current state of changes are visible here: https://github.com/llvm-mirror/llvm/compare/master...harlanhaskins:debug-info-api > > Thanks! > > Harlan Haskins > @harlanhaskins > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
Is this for the development head, or some release version of llvm? On 04/16/2017 09:32 PM, Harlan Haskins via llvm-dev wrote:> Hi llvm-dev, > > I’ve put together an implementation of a set of C API bindings for the DIBuilder interface. This builds off of work the Rust team did to wrap the DebugInfo API for rustc. It’s been expanded from their initial implementation and slightly generalized. > > I’d like to get some feedback before I submit the patch to Phabricator. Specifically, I’d like to know what I should add to it and what I should potentially leave out. I’d also like to know if this has a good chance of being accepted, or if there’s a reason I didn’t know why it wouldn’t. > > The current state of changes are visible here: https://github.com/llvm-mirror/llvm/compare/master...harlanhaskins:debug-info-api > > Thanks! > > Harlan Haskins > @harlanhaskins > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-- Rodney Bates rodney.m.bates at acm.org