Re: [PATCH] Fix wrong SRA with VIEW_CONVERT_EXPR and reverse SSO

2022-05-13 Thread Eric Botcazou via Gcc-patches
> OK. Possibly also qualifies for the branch(es) as wrong-code fix. Thanks. It's not a regression, but I can indeed put in on recent branches. -- Eric Botcazou

Re: [PATCH] Fix wrong SRA with VIEW_CONVERT_EXPR and reverse SSO

2022-05-13 Thread Richard Biener via Gcc-patches
On Fri, May 13, 2022 at 9:57 AM Eric Botcazou via Gcc-patches wrote: > > Hi, > > most cases of VIEW_CONVERT_EXPRs involving reverse scalar storage order are > disqualified for SRA because they are storage_order_barrier_p, but you can > still have a VIEW_CONVERT_EXPR to a regular composite type

[PATCH] Fix wrong SRA with VIEW_CONVERT_EXPR and reverse SSO

2022-05-13 Thread Eric Botcazou via Gcc-patches
Hi, most cases of VIEW_CONVERT_EXPRs involving reverse scalar storage order are disqualified for SRA because they are storage_order_barrier_p, but you can still have a VIEW_CONVERT_EXPR to a regular composite type being applied to a component of a record type with reverse scalar storage order,