This is an archive of the discontinued LLVM Phabricator instance.

[bindings/go] Add DebugLoc parameter to InsertXXXAtEnd()
ClosedPublic

Authored by ben-clayton on Sep 12 2018, 2:53 AM.

Details

Summary

These functions previously passed nil for the location, which always resulted in a crash.

This is a signature breaking change, but I cannot see how they could have been used before.

Diff Detail

Repository
rL LLVM

Event Timeline

ben-clayton created this revision.Sep 12 2018, 2:53 AM
pcc accepted this revision.Sep 13 2018, 1:41 PM
pcc added a subscriber: pcc.

LGTM

Signature changes are fine for this package, the API guarantees are the same as for the C bindings: https://llvm.org/docs/DeveloperPolicy.html#c-api-changes

This revision is now accepted and ready to land.Sep 13 2018, 1:41 PM
This revision was automatically updated to reflect the committed changes.