On Fri, May 7, 2021 at 12:42 PM Eric Botcazou <botca...@adacore.com> wrote:
>
> Hi,
>
> I didn't really think of pointer fields (nor of vector fields originally) when
> implementing the scalar_storage_order attribute, so they are swapped as well.
> As Ulrich pointed out, this is problematic to describe in DWARF and probably
> not very useful in any case, so the attached patch pulls them out.
>
> Tested on x86-64/Linux, OK for mainline?

OK.

Richard.

>
> 2021-05-07  Eric Botcazou  <ebotca...@adacore.com>
>
>         * doc/extend.texi (scalar_storage_order): Mention effect on pointer
>         and vector fields.
>         * tree.h (reverse_storage_order_for_component_p): Return false if
>         the type is a pointer.
> c/
>         * c-typeck.c (build_unary_op) <ADDR_EXPR>: Do not issue an error
>         on the address of a pointer field in a record with reverse SSO.
>
>
> 2021-05-07  Eric Botcazou  <ebotca...@adacore.com>
>
>         * gcc.dg/sso-12.c: New test.
>
> --
> Eric Botcazou

Reply via email to