[PATCH] D19181: Map Attribute in the C API.

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 21:17:44 PDT 2016


jyknight added a comment.

@whitequark: I think you have a picture in your mind of what the final state will look like that could better justify the choices made in this patch, than the couple of APIs added here do. And I'm willing to imagine that it could be better than my proposal. But, without a complete concrete API proposal being written down, it's not easy to come to that shared understanding.

So, like I said before to @deadalnix a while ago, I'd really like to see the complete proposed API for attribute access, even if an initial code patch doesn't implement the whole thing yet. Can one of you write that up, similar to how I did?


http://reviews.llvm.org/D19181





More information about the llvm-commits mailing list