rsmith added a comment.

I have no philosophical objection to exposing this, but defer to @rjmccall on 
that. As is usual with our C++ API, there's no guarantee that we will keep this 
working in future versions. (Eg, if we move to an IR representation using 
something non-GEPable to represent a source-level class type, we would remove 
this function as it would be meaningless.)

This interface does not seem to be sufficient to support bit-fields. You should 
also indicate *which* record layout (complete object type or base subobject 
type) the field number is for. I don't think there's any guarantee that the 
same indexes work in both.


https://reviews.llvm.org/D38473



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to