chill added a comment.

In https://reviews.llvm.org/D46013#1140336, @t.p.northover wrote:

> I'm fine with the ABI changes, but I'm not very convinced by the 
> "NaturalAlignment" name.


I'd rather not put target names in API functions. The meaning of that field is 
pretty target
independent ("alignment of type, before alignment adjustments")
If we use it only in ARM/AArch64 we can put a comment in that sense and
maybe rename it to `UnadjustedAlignement` or something else more descriptive 
than `NaturalAlignement` ?


https://reviews.llvm.org/D46013



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

Reply via email to