This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, PR 50622] Force a gimple operand in load_assign_lhs_subreplacements when necessary
- From: Jeff Law <law at redhat dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Guenther <rguenther at suse dot de>
- Date: Thu, 01 Dec 2011 10:37:36 -0700
- Subject: Re: [PATCH, PR 50622] Force a gimple operand in load_assign_lhs_subreplacements when necessary
- References: <20111201161524.GA4447@virgil.arch.suse.de>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 12/01/11 09:15, Martin Jambor wrote:
> Hi,
>
> PR 50622 is an omission in load_assign_lhs_subreplacements, which
> should force a gimple operand on a RHS of a gimple assignment if
> both sides are new replacements of scalar types which are not
> gimple registers, because they are partially modified (which can
> happen to complex numbers and bit-fields).
>
> Fixed with the patch below. It passes bootstrap and testsuite on
> x86_64-linux, I am about to do the same on the 4.6 branch because
> I'd like to commit it there as well. OK for trunk and the 4.6
> branch?
>
> Thanks,
>
> Martin
>
>
> 2011-12-01 Martin Jambor <mjambor@suse.cz>
>
> PR tree-optimization/50622 * tree-sra.c
> (load_assign_lhs_subreplacements): Force gimple operand if both
> lacc and racc are grp_partial_lhs.
>
> * testsuite/g++.dg/tree-ssa/pr50622.C: New test.
OK for trunk. Release manager owns 4.6 branch.
jeff
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEcBAEBAgAGBQJO17tgAAoJEBRtltQi2kC7O+AIAIPeCZFGJulh6WtuF3KM5zDM
C9cibIUI0XjeOB9CpuFSXf3+vbzXkbsiaS/E+E5x6TxOQEsHJJ3Z6SrX342t9OEO
hPSm0vKoFDUV5vfc9nUhb/CsibgbmDL2rcAZO1rT6mvwzyZIdSR3kVGLTueSZIpC
NSLHd+p3yaYcTzUlukSCxK3fphPWhqso4/0bg8Eq7+yZE56AprbTVg9n/fHmocrB
VmlOHriOsI18uAyUOhleT2mDn0x64sV8jS0lMb4sgIg3cguUYqyqCI8C4lGadaFL
k6H/0gnky7zlH8d8ZICAYBzC86vLCBS6wxNTQjvsw0C/2P7e7Wl/fOi7GYS1k4w=
=SwO7
-----END PGP SIGNATURE-----