This is an archive of the discontinued LLVM Phabricator instance.

[LLVM-C] Publicly expose getters of MetadataType, TokenType
ClosedPublic

Authored by CodaFi on Oct 11 2017, 11:02 AM.

Details

Summary

Expose getters for MetadataType and TokenType publicly in the C API. Discovered a need for these while trying to wrap the intrinsics API.

Diff Detail

Repository
rL LLVM

Event Timeline

CodaFi created this revision.Oct 11 2017, 11:02 AM
echristo edited edge metadata.

Adding in more reviewers. In general I'm fine here, but since they've been in this area much more I'd like to hear from them.

whitequark requested changes to this revision.Oct 13 2017, 11:25 PM
whitequark added inline comments.
include/llvm-c/Core.h
1155 ↗(On Diff #118643)

We don't add any new functions that operate on global context anymore.

This revision now requires changes to proceed.Oct 13 2017, 11:25 PM
CodaFi updated this revision to Diff 119022.Oct 14 2017, 8:19 AM
CodaFi edited edge metadata.
CodaFi marked an inline comment as done.
whitequark accepted this revision.Oct 15 2017, 12:04 AM
This revision is now accepted and ready to land.Oct 15 2017, 12:04 AM

Ping @whitequark for a merge. Looks like @deadalnix hasn't been active for a while.

This revision was automatically updated to reflect the committed changes.