ojhunt wrote:

This approach of having the record layout test the alignment of fields 
internally is wrong. The field types have their alignment. Your fix should be 
to ensure that the types have correctly reported alignment, not to have record 
layout behaviour change on the basis of specific field types.

You should not need to have vector or float80 specific hacks, those types 
should simply report the correct alignment. If they report the correct 
alignment requirements the rest of the behaviour should fall out. If it does 
not then either something else is wrong in codegen or the presumption in the 
test case is wrong.

https://github.com/llvm/llvm-project/pull/208256
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to