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

[Amaury SECHET via llvm-commits]

deadalnix added a comment.

To be able to map the different kind of attribute, one does need to expose LLVMAttributeRef or alike. You proposal doesn't allow for this.

On exposing the AttributeSet, I'd rather avoid exposing something that is not strictly necessary as it may end up evolve in the future. In addition, a ton of code wrap AttributeSet and use setter/getter in the C++ codebase, which should be a sign that this isn't such a great abstraction APIwise.


http://reviews.llvm.org/D19181