This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: [4.5 Regression] RPR c/40172: evision 147596 breaks bootstrap
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Manuel López-Ibáñez <lopezibanez at gmail dot com>, Simon Martin <simartin at users dot sourceforge dot net>, Gcc Patch List <gcc-patches at gcc dot gnu dot org>, "Joseph S. Myers" <joseph at codesourcery dot com>, Mark Mitchell <mark at codesourcery dot com>, Jason Merrill <jason at redhat dot com>, Nathan Sidwell <nathan at codesourcery dot com>, Vladimir Makarov <vmakarov at redhat dot com>
- Date: Tue, 19 May 2009 10:35:01 +0200
- Subject: Re: PATCH: [4.5 Regression] RPR c/40172: evision 147596 breaks bootstrap
- References: <6dc9ffc80905181801u328d31ads7d39bb417dfabe88@mail.gmail.com>
On Tue, May 19, 2009 at 3:01 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, May 18, 2009 at 5:36 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> On Sun, May 17, 2009 at 1:21 PM, Richard Guenther
>> <richard.guenther@gmail.com> wrote:
>>> On Sat, May 16, 2009 at 5:17 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>> 2009/5/16 Manuel López-Ibáñez <lopezibanez@gmail.com>:
>>>>> That is strange, it should not warn for constants. Is it really 0 or
>>>>> something else?
>>>>
>>>>
>>>> I open a bug report with a testcase:
>>>>
>>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40172
>>>
>>> Given all this and that bootstrap is still not fixed for some targets
>>> I seriously
>>> doubt this warning is worth all this trouble.
>>>
>>> Thus, I suggest to revert the patch or make the warning not part of -Wall.
>>>
>>
>> How about this patch?
>>
>> --
>> H.J.
>> ---
>> gcc/
>>
>> 2009-05-18 ?Manuel López-Ibáñez ?<manu@gcc.gnu.org>
>
> Oops. Wrong ChangeLog.
This kind of stuff totally sucks.
Richard.
>
> H.J.
> ---
> gcc/
>
> 2009-05-18 ?H.J. Lu ?<hongjiu.lu@intel.com>
>
> ? ? ? ?PR c/40172
> ? ? ? ?* toplev.c (process_options): Avoid
> ? ? ? ?if (!FRAME_GROWS_DOWNWARD && flag_stack_protect).
>
> gcc/testscase/
>
> 2009-05-18 ?H.J. Lu ?<hongjiu.lu@intel.com>
>
> ? ? ? ?PR c/40172
> ? ? ? ?* gcc.dg/pr40172.c: Add more tests.
>
>
> --
> H.J.
>