https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77728

--- Comment #16 from Ramana Radhakrishnan <ramana at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #10)
> aarch64_function_arg_alignment
> has:
>   for (tree field = TYPE_FIELDS (type); field; field = DECL_CHAIN (field))
>     alignment = std::max (alignment, DECL_ALIGN (field));
> and thus has similar problem (no testcases in this case though).

A prototype fix (needs tidying up for sure!) that warns for aarch64 is as
attached. There are no regressions on a full bootstrap and test run. I would be
grateful for any comments.

Note that I haven't managed to actually trigger an ABI warning because of this
on aarch64. 

Note that this is dependent on clarification with respect to the AAPCS . 

I'm not yet happy with the amount of refactoring we have to do to get the
warnings on ARM.


regards
Ramana

Reply via email to