[PATCH] D19037: [llvm-c] Add the ability to create debug locations

[Amaury SECHET via llvm-commits]

deadalnix added a comment.

Sorry, missed that bit. OK, let's discuss about how to proceed with this, as it obviously has an impact on this patch.

It seems to me that mapping the new DI API is a good opportunity to introduce LLVMMetadataRef and the MD class hierarchy. Wrapping everything in MetadataAsValue is going to create a tremendous amount of waste. I submitted http://reviews.llvm.org/D19088 , which isn't a a state where it is ready to be merged, but that should help the discussion here.


http://reviews.llvm.org/D19037