This is an archive of the discontinued LLVM Phabricator instance.

Expose setPersonalityFn to Go
ClosedPublic

Authored by axw on Jul 10 2015, 7:52 PM.

Details

Summary

Add Value.SetPersonality to the Go bindings. The Go
bindings' Builder.CreateLandingPad has been updated,
removing the obsolete personality argument.

Background

The personality attribute was removed from LandingPadInst
in r239940, and llvm::Function::setPersonalityFn introduced.

There was no corresponding change to either the C API or
Go bindings. The Go bindings were broken until r239940, but
that change was just to ignore the personality argument.
This broke llgo.

Diff Detail

Event Timeline

axw updated this revision to Diff 29507.Jul 10 2015, 7:52 PM
axw retitled this revision from to Expose setPersonalityFn to C and Go.
axw updated this object.
axw added reviewers: pcc, majnemer.
axw added a subscriber: llvm-commits.
pcc edited edge metadata.Jul 12 2015, 5:57 PM

D10946 makes a similar change to the C API. It's ready to land, so you could commit it and update this change to use it instead.

You need the getter as well.

axw added a comment.Jul 13 2015, 3:37 AM

You need the getter as well.

Not foot what I'm doing, but yes, it should probably have both. I've accepted your rev, land away and I'll update the Go bindings here. Thanks.

axw updated this revision to Diff 29664.Jul 14 2015, 4:45 AM
axw edited edge metadata.

Update to base on top of D10946

axw updated this object.Jul 14 2015, 4:46 AM
axw retitled this revision from Expose setPersonalityFn to C and Go to Expose setPersonalityFn to Go.
pcc accepted this revision.Jul 15 2015, 4:40 AM
pcc edited edge metadata.

LGTM

This revision is now accepted and ready to land.Jul 15 2015, 4:40 AM
axw closed this revision.Jul 15 2015, 6:03 AM