Re: [PATCH] Fix SRA and initializers

[Andreas Jaeger <aj at suse dot de>]

Andrew Pinski <pinskia@physics.uc.edu> writes:

> I noticed when you have a complex initializer in a constant decl
> and we decided to SRA the copy, we need to gimplify the initializer.
> Since gimplify_stmt does not mark the temporary variables for renaming,
> we get an ICE because the variables are not renamed.  The easy way and
> correct way to fix this is to change over to use force_gimple_operand
> instead of gimplify_stmt.
>
> This fixes the Ada bootstrap failure on powerpc-darwin.
>
> OK? Bootstrapped and tested on powerpc-darwin with no regressions.

This patch seems to break gcc on Linux/ia64 and Linux/x86_64, with
that patch applied, we get the following ICE compiling rcs:

gcc -c  -Dhas_conf_h  -I. -I. -O2 -fmessage-length=0 -Wall ci.c
ci.c: In function 'removelock':
ci.c:1139: warning: suggest explicit braces to avoid ambiguous 'else'
ci.c: In function 'getlogmsg':
ci.c:1254: internal compiler error: Segmentation fault
Please submit a full bug report,
with preprocessed source if appropriate.


Andreas
-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
  SUSE Linux Products GmbH, Maxfeldstr. 5, 90409 NÃrnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

Attachment: pgp00000.pgp
Description: PGP signature