[PATCH] D114860: [llvm-c] Make LLVMAddAlias opaque pointer compatible

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

I'd also like some advice on deprecations in the C API: As this is an FFI API, marking the functions as deprecated in the header is of limited usefulness, and we currently don't do that. But this means that people using the FFI API are likely not aware they need to take action wrt opaque pointers. I would propose that we go ahead and delete old deprecated functions like LLVMBuildInvoke, so that people are forced to take action now, while easy workarounds (like LLVMGetElementType) are still available.

The only alternative that comes to mind is a link-time deprecation, but I'm not familiar with how these are done and how well they work across platforms.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114860/new/

https://reviews.llvm.org/D114860