[PATCH] D47179: [LLVM-C] Add Bindings For Named Metadata

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

In https://reviews.llvm.org/D47179#1108484, @dexonsmith wrote:

> In https://reviews.llvm.org/D47179#1107993, @aprantl wrote:
>
> > Adding Duncan to the review list who (presumably) added the TODO and may have some thoughts on this.
>
>
> Making `NamedMDNode` inherit from `Metadata` will allow `MDNode` to reference `NamedMDNode`.
>
> - What's the use case for this?
> - What complexity does it add to the IR?  (Especially, during serialization and LTO...)
>
>   Perhaps this is worth doing, but we need a compelling reason.  In retrospect, I'm not sure I had sufficient motiviation when I added the TODO.


If the goal is just to expose them to the C API, you can expose them as is by adding LLVMNamedMetadataRef, etc.


Repository:
  rL LLVM

https://reviews.llvm.org/D47179