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