Page MenuHomePhabricator

[opaque pointer types] Update InvokeInst creation APIs to consistently accept a callee-type argument.

Authored by jyknight on Jan 10 2019, 1:22 PM.

Diff Detail


Event Timeline

jyknight created this revision.Jan 10 2019, 1:22 PM
jyknight updated this revision to Diff 181153.Jan 10 2019, 2:07 PM

(Ran clang format; forgot to do before uploading before)

whitequark requested changes to this revision.Jan 10 2019, 7:35 PM
whitequark added inline comments.
3411 ↗(On Diff #181153)

Please add NameLen as this is the convention for all new C APIs.

This revision now requires changes to proceed.Jan 10 2019, 7:35 PM
whitequark added inline comments.Jan 10 2019, 7:40 PM
3411 ↗(On Diff #181153)

(See discussion over at

niravd accepted this revision.Jan 14 2019, 12:09 PM
niravd added a subscriber: niravd.


This revision was not accepted when it landed; it landed in state Needs Revision.Jan 14 2019, 1:41 PM
This revision was automatically updated to reflect the committed changes.