[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[jakub at gcc dot gnu dot org]

--- Comment #22 from jakub at gcc dot gnu dot org  2009-03-30 13:51 ---
What exact regression was fixed by the
http://gcc.gnu.org/viewcvs?root=gccview=revrev=145102
commit?


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[jakub at gcc dot gnu dot org]

--- Comment #23 from jakub at gcc dot gnu dot org  2009-03-30 17:42 ---
Subject: Bug 35652

Author: jakub
Date: Mon Mar 30 17:42:27 2009
New Revision: 145308

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=145308
Log:
Revert PR c++/35652

Removed:
branches/gcc-4_4-branch/gcc/testsuite/g++.dg/warn/pr35652.C
branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/pr35652.c
Modified:
branches/gcc-4_4-branch/gcc/ChangeLog
branches/gcc-4_4-branch/gcc/builtins.c
branches/gcc-4_4-branch/gcc/c-common.c
branches/gcc-4_4-branch/gcc/c-common.h
branches/gcc-4_4-branch/gcc/c-typeck.c
branches/gcc-4_4-branch/gcc/cp/ChangeLog
branches/gcc-4_4-branch/gcc/cp/typeck.c
branches/gcc-4_4-branch/gcc/testsuite/ChangeLog
branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/format/plus-1.c


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[dominiq at lps dot ens dot fr]

--- Comment #17 from dominiq at lps dot ens dot fr  2009-03-27 14:30 ---
The commit in comment #16 breaks bootstrap on *-darwin9:

...
cc1: warnings being treated as errors
../../gcc-4.4-work/gcc/c-common.c: In function 'pointer_int_sum':
../../gcc-4.4-work/gcc/c-common.c:3320: error: format '%ld' expects type 'long
int', but argument 4 has type 'long long int'
...


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[rguenth at gcc dot gnu dot org]

--- Comment #18 from rguenth at gcc dot gnu dot org  2009-03-27 14:34 
---
If this isn't resolved quickly please open a bugreport for this.


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[dominiq at lps dot ens dot fr]

--- Comment #19 from dominiq at lps dot ens dot fr  2009-03-27 14:44 ---
 If this isn't resolved quickly please open a bugreport for this.

I'll do it.  This is a recurrent issue due to maintainers not using the -Werror
in their tests.
Unfortunately I don't remember how such failures have been fixed in the past (a
cast to long int?).


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[dominiq at lps dot ens dot fr]

--- Comment #20 from dominiq at lps dot ens dot fr  2009-03-27 14:50 ---
Currently testing the patch in
http://gcc.gnu.org/ml/gcc-patches/2009-03/msg01377.html.


-- 


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

[Bug c++/35652] [4.2/4.3 Regression] offset warning should be given in the front-end

[dominiq at lps dot ens dot fr]

--- Comment #21 from dominiq at lps dot ens dot fr  2009-03-27 15:13 ---
The failure reported in comment #17 has been fixed by revision 145109.


-- 


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