This is an archive of the discontinued LLVM Phabricator instance.

[LLVM-C] Match C-API integer types to the underlying used CPP-API
AbandonedPublic

Authored by ekatz on Apr 23 2020, 12:13 PM.

Details

Summary

Fix PR9242

Diff Detail

Event Timeline

ekatz created this revision.Apr 23 2020, 12:13 PM

This is a fairly substantial ABI break not sure how to deal with this best.

ekatz added a comment.Apr 28 2020, 5:47 AM

This is a fairly substantial ABI break not sure how to deal with this best.

Shouldn't users include the headers in their project?

CodaFi requested changes to this revision.May 1 2020, 5:01 PM

We are committed to ABI stability in the core bindings. If you wish to pursue this change, you will need to seek consensus on the mailing list. I'm personally not opposed to an ABI break, but it *must* be done in the open.

This revision now requires changes to proceed.May 1 2020, 5:01 PM
ekatz abandoned this revision.May 1 2020, 10:04 PM

Thanks for the references.
I'm abandoning this change, due to lack of interest.