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

2009-03-27 Thread andreasmeier80 at gmx dot de
--- Comment #15 from andreasmeier80 at gmx dot de 2009-03-27 07:16 --- Approved here: http://gcc.gnu.org/ml/gcc-patches/2009-03/msg01079.html -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35652

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

2009-02-08 Thread manu at gcc dot gnu dot org
--- Comment #14 from manu at gcc dot gnu dot org 2009-02-08 15:46 --- Patch: http://gcc.gnu.org/ml/gcc-patches/2009-02/msg00285.html -- manu at gcc dot gnu dot org changed: What|Removed |Added

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

2009-02-03 Thread bonzini at gnu dot org
--- Comment #13 from bonzini at gnu dot org 2009-02-03 16:40 --- I agree that the patch is correct *without* the -1, so... ping :) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35652

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

2008-11-02 Thread rguenther at suse dot de
--- Comment #9 from rguenther at suse dot de 2008-11-02 12:20 --- Subject: Re: [4.2/4.3/4.4 Regression] offset warning should be given in the front-end On Sat, 1 Nov 2008, manu at gcc dot gnu dot org wrote: --- Comment #8 from manu at gcc dot gnu dot org 2008-11-01 17:44

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

2008-11-02 Thread manu at gcc dot gnu dot org
--- Comment #10 from manu at gcc dot gnu dot org 2008-11-02 12:53 --- (In reply to comment #9) This is my current patch and it works in this testcase. However, it also triggers on cases like: const char *p = str + sizeof(str) Perhaps I am doing this at the wrong place. Any

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

2008-11-02 Thread rguenth at gcc dot gnu dot org
--- Comment #11 from rguenth at gcc dot gnu dot org 2008-11-02 13:02 --- I'm not sure. Does TREE_STRING_LENGTH in the particular case include the NULL character? Does sizeof(str) include the NULL character? In principle it is allowed to have a pointer point one after the last

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

2008-11-02 Thread Andrew Thomas Pinski
Sent from my iPhone On Nov 2, 2008, at 4:53 AM, manu at gcc dot gnu dot org [EMAIL PROTECTED] wrote: --- Comment #10 from manu at gcc dot gnu dot org 2008-11-02 12:53 --- (In reply to comment #9) This is my current patch and it works in this testcase. However, it also

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

2008-11-02 Thread pinskia at gmail dot com
--- Comment #12 from pinskia at gmail dot com 2008-11-02 13:07 --- Subject: Re: [4.2/4.3/4.4 Regression] offset warning should be given in the front-end Sent from my iPhone On Nov 2, 2008, at 4:53 AM, manu at gcc dot gnu dot org [EMAIL PROTECTED] wrote: --- Comment #10

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

2008-11-01 Thread manu at gcc dot gnu dot org
--- Comment #8 from manu at gcc dot gnu dot org 2008-11-01 17:44 --- This is my current patch and it works in this testcase. However, it also triggers on cases like: const char *p = str + sizeof(str) Perhaps I am doing this at the wrong place. Any suggestions? @@ -3322,10 +3323,36 @@

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

2008-10-30 Thread manu at gcc dot gnu dot org
--- Comment #7 from manu at gcc dot gnu dot org 2008-10-30 22:44 --- The location info seems to be fixed in mainline. Nonetheless, I agree completely with Richard. This warning belongs in the front-end. Moreover we fail to diagnose: const char *s = 'x' + y; in both C and C++. Also,