This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C/C++/IRA] PR 16302 gcc fails to warn about some common logic errors
- From: Manuel López-Ibáñez <lopezibanez at gmail dot com>
- To: Simon Martin <simartin at users dot sourceforge dot net>
- Cc: 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: Sat, 16 May 2009 11:44:27 +0200
- Subject: Re: [C/C++/IRA] PR 16302 gcc fails to warn about some common logic errors
- References: <6c33472e0905142348y142c319br6d5ca874ec0456e7@mail.gmail.com> <4A0E7EC6.9020003@users.sourceforge.net>
That is strange, it should not warn for constants. Is it really 0 or
something else?
Cheers,
Manuel.
2009/5/16 Simon Martin <simartin@users.sourceforge.net>:
> Hello,
>
> For information, following this, bootstrap is broken on darwin; see
> http://gcc.gnu.org/ml/gcc-patches/2009-05/msg00974.html
>
> Best regards,
> Simon
>
> Manuel López-Ibáńez wrote:
>>
>> Bootstrapped and regression tested on x86_64-unknown-linux-gnu with
>> --enable-languages=all
>>
>> OK for trunk?
>>
>> 2009-05-15 ?Manuel Lopez-Ibanez ?<manu@gcc.gnu.org>
>>
>> ? ? ? ?PR 16302
>> ? ? ? ?* ira-conflicts.c (add_insn_allocno_copies): Fix wrong
>> ? ? ? ?conditional.
>> ? ? ? ?* fold-const.c (make_range,build_range_check,merge_ranges): Move
>> ? ? ? ?declaration to...
>> ? ? ? ?(merge_ranges): Returns bool.
>> ? ? ? ?* tree.h (make_range): .. to here.
>> ? ? ? ?(build_range_check): Likewise.
>> ? ? ? ?(merge_ranges): Likewise. Renamed from merge_ranges.
>> ? ? ? ?* c-typeck.c (parser_build_binary_op): Update calls to
>> ? ? ? ?warn_logical_operator.
>> ? ? ? ?* c-common.c (warn_logical_operator): Add new warning.
>> ? ? ? ?* c-common.h (warn_logical_operator): Update declaration.
>> cp/
>> ? ? ? ?* call.c (build_new_op): Update calls to warn_logical_operator.
>>
>> testsuite/
>> ? ? ? ?* gcc.dg/pr16302.c: New.
>> ? ? ? ?* g++.dg/warn/pr16302.C: New.
>>
>
>
>
>