diff options
author | Eric Botcazou <ebotcazou@adacore.com> | 2022-05-13 11:15:08 +0200 |
---|---|---|
committer | Eric Botcazou <ebotcazou@adacore.com> | 2022-05-13 11:19:01 +0200 |
commit | 7b844206ec51931d0a81aa4e4fa3b62f6ecef0ca (patch) | |
tree | 13319be8c4b929b89715cafc316236e6273a7800 /gcc/ada/gcc-interface | |
parent | c77e4873b14bd84fc6715db8980ce14886b28906 (diff) | |
download | gcc-7b844206ec51931d0a81aa4e4fa3b62f6ecef0ca.zip gcc-7b844206ec51931d0a81aa4e4fa3b62f6ecef0ca.tar.gz gcc-7b844206ec51931d0a81aa4e4fa3b62f6ecef0ca.tar.bz2 |
Fix wrong SRA with VIEW_CONVERT_EXPR and reverse SSO
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.
In this case the bypass for !useless_type_conversion_p in sra_modify_assign,
albeit already heavily guarded, triggers and may generate wrong code, so the
patch makes sure that it does only when the SSO is the same on both side.
gcc/
* tree-sra.cc (sra_modify_assign): Check that scalar storage order
is the same on the LHS and RHS before rewriting one with the model
of the other.
gcc/testsuite/
* gnat.dg/sso17.adb: New test.
Diffstat (limited to 'gcc/ada/gcc-interface')
0 files changed, 0 insertions, 0 deletions