This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Improve Store CCP for NULL DECL_INITIAL (PR 35402)

From: "H.J. Lu" <hjl dot tools at gmail dot com>
To: "Richard Guenther" <richard dot guenther at gmail dot com>
Cc: Andrew_Pinski at playstation dot sony dot com, gcc-patches at gcc dot gnu dot org, "Ye, Joey" <joey dot ye at intel dot com>, weiliang dot lin at intel dot com
Date: Thu, 6 Mar 2008 22:08:17 -0800
Subject: Re: [PATCH] Improve Store CCP for NULL DECL_INITIAL (PR 35402)
References: <OF1A878EA6.2E17D694-ON88257404.007666FF-88257404.0077BF31@playstation.sony.com> <84fc9c000803061352m17ee74d7sf8444ce4fa980be3@mail.gmail.com>

This patch caused:

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35494

H.J.
On Thu, Mar 6, 2008 at 1:52 PM, Richard Guenther
<richard.guenther@gmail.com> wrote:
> On Thu, Mar 6, 2008 at 10:48 PM,  <Andrew_Pinski@playstation.sony.com> wrote:
>  > Hi,
>  >   When the DECL_INITIAL is NULL for read only variables, we don't
>  >  propagate the default (zero) value to the code so we still reference the
>  >  symbol.
>  >  This patch changes Store CPP to do this optimization for these variables.
>  >  It only handles intergal and scalar float types because fold_convert does
>  >  not handle many more types.
>  >
>  >  OK?  Bootstrapped and tested on i686-linux-gnu with no regressions.
>
>  Ok.
>
>  Thanks,
>  Richard.
>

References:
- [PATCH] Improve Store CCP for NULL DECL_INITIAL (PR 35402)
  - From: Andrew_Pinski
- Re: [PATCH] Improve Store CCP for NULL DECL_INITIAL (PR 35402)
  - From: Richard Guenther

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]