Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-17 Thread H.J. Lu
On Wed, Jul 6, 2011 at 11:25 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Jul 6, 2011 at 6:26 AM, Richard Guenther rguent...@suse.de wrote: This fixes PR49645 - with MEM_REF the value-numbering machinery to look through aggregate copies wasn't working reliably as we have two representations

[PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Richard Guenther
This fixes PR49645 - with MEM_REF the value-numbering machinery to look through aggregate copies wasn't working reliably as we have two representations for X, X and MEM[X]. The following patch fixes that by internally always using the more complicated representation. The patch needs consistent

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Michael Matz
Hi, On Wed, 6 Jul 2011, Richard Guenther wrote: *** copy_reference_ops_from_ref (tree ref, V *** 579,585 memset (temp, 0, sizeof (temp)); /* We do not care for spurious type qualifications. */ ! temp.type = TYPE_MAIN_VARIANT (TREE_TYPE (ref));

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Joseph S. Myers
On Wed, 6 Jul 2011, Richard Guenther wrote: * c-decl.c (finish_decl): Also set DECL_HARD_REGISTER for global register variables. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread H.J. Lu
On Wed, Jul 6, 2011 at 6:26 AM, Richard Guenther rguent...@suse.de wrote: This fixes PR49645 - with MEM_REF the value-numbering machinery to look through aggregate copies wasn't working reliably as we have two representations for X, X and MEM[X].  The following patch fixes that by internally