This is an archive of the discontinued LLVM Phabricator instance.

[llgo] drop debug/DIBuilder.Declare
ClosedPublic

Authored by axw on Aug 9 2015, 6:40 PM.

Details

Summary

Companion to D11864, which updates the
DIBuilder bindings to match a recent
change in the underlying API.

We drop the debug/DIBuilder.Declare method
for now, which is unused, as otherwise it will
be broken with the new DIBuilder API in the
Go bindings.

Diff Detail

Event Timeline

axw updated this revision to Diff 31627.Aug 9 2015, 6:40 PM
axw retitled this revision from to [llgo] update to new DIBuilder API.
axw updated this object.
axw added a reviewer: pcc.
axw added a subscriber: llvm-commits.
pcc edited edge metadata.Aug 17 2015, 12:15 AM

Is this code used at all at the moment? It doesn't look like it is, so maybe we should remove it?

axw updated this revision to Diff 33314.Aug 27 2015, 5:18 AM
axw edited edge metadata.

Delete unused Declare method

axw added a comment.Aug 27 2015, 5:18 AM
In D11891#225503, @pcc wrote:

Is this code used at all at the moment? It doesn't look like it is, so maybe we should remove it?

Yes, done. I'll resurrect it if/when we need it.

axw added a comment.Aug 31 2015, 8:09 AM

@pcc Seeing as you suggested the alternative change, I'll assume this is good to land.

axw updated this revision to Diff 33581.Aug 31 2015, 8:10 AM

Rebase

axw retitled this revision from [llgo] update to new DIBuilder API to [llgo] drop debug/DIBuilder.Declare.Aug 31 2015, 8:12 AM
axw updated this object.
pcc accepted this revision.Aug 31 2015, 1:09 PM
pcc edited edge metadata.

LGTM

This revision is now accepted and ready to land.Aug 31 2015, 1:09 PM
axw closed this revision.Sep 1 2015, 4:53 AM