[PATCH] D19641: Add an alternative to LLVMMDNode that will only accept metadata as argument.

Amaury SECHET via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 19:59:12 PDT 2016


deadalnix added a comment.

@jyknight Historically, Metadata were some kind of Value. But now, Metadata has its own hierarchy. In order to expose this in the C API, we need a transition plan.

The main problem here is that LLVMMDNode accept both metadata and values. Because of this, it is impossible to split the 2 right now. In order to get out of there, I want to go through a deprecation cycle for LLVMMDNode in favor of an alternative that will only accept metadata.

Right now, these metadata are wrapped as value, but passing down anything else than a metadata result in an error. As a result, it is possible to switch to LLVMMetadataRef the method that produce and consume metadata at once in such a way that thing still works with outdated headers.


http://reviews.llvm.org/D19641





More information about the llvm-commits mailing list