--- Comment #25 from dominiq at lps dot ens dot fr 2010-02-18 08:00 ---
> Would you mind testing the attached patch?
Apparently the patch in comment #23 does not fix the problem (incremental
update of gcc and partial test):
make -k check-gfortran RUNTESTFLAGS="dg.exp=complex*.f90
--tar
--- Comment #1 from jakub at gcc dot gnu dot org 2010-02-18 08:56 ---
You just misunderstand it. You'd need a 128-bit int __attribute__((mode (TI)))
value if you want it to mean a pair, because for -m64 registers are 64-bit.
If you use smaller precision type, "A" constraint acts just as
--- Comment #1 from burnus at gcc dot gnu dot org 2010-02-18 09:25 ---
Why is this marked as 4.4 regression? According to my regression hunt it fails
since the patch http://gcc.gnu.org/viewcvs?view=revision&revision=156749 which
has only be committed to the trunk. I just checked and the
--- Comment #17 from paolo dot carlini at oracle dot com 2010-02-18 09:36
---
(In reply to comment #16)
> I don't understand why all libstdc++ headers should not have #pragma GCC
> system_header in them. Would you please explain that?
Hi Mark. For sure all the *library proper* headers
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jakub at gcc dot gnu dot org
|dot org
--- Comment #28 from paolo dot carlini at oracle dot com 2010-02-18 09:42
---
Thanks Gaby, eventually I learned something ;)
Hopefully the remaining correctness issues can be fixed quickly in the C++
front-end, honestly before this PR I wasn't aware of any correctness regression
in thi
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-02-18 09:43 ---
Yep. Fixed. Thanks Ira.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #29 from rguenth at gcc dot gnu dot org 2010-02-18 09:46
---
(In reply to comment #28)
> Thanks Gaby, eventually I learned something ;)
>
> Hopefully the remaining correctness issues can be fixed quickly in the C++
> front-end, honestly before this PR I wasn't aware of any
--- Comment #4 from jakub at gcc dot gnu dot org 2010-02-18 09:48 ---
Created an attachment (id=19904)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19904&action=view)
gcc45-pr43107.patch
Fix, tested so far just on this testcase.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--- Comment #30 from paolo dot carlini at oracle dot com 2010-02-18 09:53
---
Argh, so the C front-end manages to be both correct and fast. We should
definitely do something about C++...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43108
--- Comment #3 from ebotcazou at gcc dot gnu dot org 2010-02-18 09:56
---
Looking into it.
--
ebotcazou at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from paolo dot carlini at oracle dot com 2010-02-18 10:00
---
Dave, what do you recommend about this issue? Does it affect cygwin too?
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
-
--- Comment #10 from paolo dot carlini at oracle dot com 2010-02-18 10:12
---
In short, I see two conceptually separate issues: 1- We run the configure tests
for those functions in GNU mode, thus if on a target an header doesn't declare
some functions in ISO strict mode, the user experi
--- Comment #2 from pault at gcc dot gnu dot org 2010-02-18 10:26 ---
(In reply to comment #1)
> Why is this marked as 4.4 regression? According to my regression hunt it fails
> since the patch http://gcc.gnu.org/viewcvs?view=revision&revision=156749 which
> has only be committed to the
--- Comment #26 from bernds_cb1 at t-online dot de 2010-02-18 11:51 ---
Created an attachment (id=19905)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19905&action=view)
A patch to help debug the problem
I'll need some help since on my system a compiler targetting
powerpc-apple-da
--- Comment #27 from dominiq at lps dot ens dot fr 2010-02-18 12:17 ---
> You'll need to use a kind of binary search using the -fdbg-cnt=rnreg:N option,
> where N is an integer. Start with something small, maybe even 1, then double
> it until the failure appears. Then do a binary searc
--- Comment #28 from bernds_cb1 at t-online dot de 2010-02-18 12:21 ---
Only when building the testcase.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42220
--- Comment #7 from jakub at gcc dot gnu dot org 2010-02-18 12:30 ---
On the 4.4 branch actually cc1 behaves differently from cc1plus.
In any case, I believe the two setne's plus testb code is actually the worst
of the options, and e.g. with -Os it is even much larger.
Just compile the t
--- Comment #8 from jakub at gcc dot gnu dot org 2010-02-18 12:47 ---
Created an attachment (id=19906)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19906&action=view)
gcc45-pr42233.patch
The reason why it behaves differently with __builtin_expect from normal
condition is that in
--- Comment #11 from davek at gcc dot gnu dot org 2010-02-18 12:54 ---
(In reply to comment #9)
> Dave, what do you recommend about this issue? Does it affect cygwin too?
>
This particular problem doesn't occur on cygwin, but we've had similar issues
in the past with pre-c99 headers on
--- Comment #12 from ramana at gcc dot gnu dot org 2010-02-18 13:13 ---
Subject: Bug 40887
Author: ramana
Date: Thu Feb 18 13:13:03 2010
New Revision: 156862
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156862
Log:
PR target/40887
Backport from trunk.
--- Comment #9 from bonzini at gnu dot org 2010-02-18 13:17 ---
I'm relieved that cond-optab is in no way related to this. :-)
I'm not expert of the gimplifier but the patch makes sense and looks good.
--
bonzini at gnu dot org changed:
What|Removed
--- Comment #5 from Rudolf dot Leitgeb at gmx dot at 2010-02-18 13:28
---
Right away: I am NOT an expert in compiler or optimizer design, so please bear
with me.
I understand, that (((unsigned char)1) << ((unsigned char)something)) may need
more than 8 bits for representation and that
testcase.cpp
template < typename >
struct A {
struct S;
A < S > ht;
};
struct B : A < B > {};
--
without -ftemplate-depth, it instantiates templates up do default depth, giving
760KiB of diagnostics for this simple testcase (for 4.5.0)
this is ou
--- Comment #13 from ramana at gcc dot gnu dot org 2010-02-18 13:34 ---
Fixed.
--
ramana at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
Overview Description:
GCC version: 4.4.1
System info: 2.6.31-14-generic #48-Ubuntu SMP Fri Oct 16 14:04:26 UTC 2009 i686
GNU/Linux, Ubuntu 9.10
GCC I used was the Ubuntu 9.10 native c++ compiler, installed using Synaptic
package manager.
The case is the following:
There are two files to be com
--- Comment #29 from dominiq at lps dot ens dot fr 2010-02-18 13:42 ---
In order to compile gcc/regrename.c, I had to add
+ enum debug_counter rnreg;
Is this right?
Then compiling the test with
gfc -fdbg-cnt=rnreg:1 -m64 -O1 -frename-registers
/opt/gcc/_gcc_clean/gcc/testsuite/gfort
--- Comment #30 from steven at gcc dot gnu dot org 2010-02-18 13:53 ---
It looks like there should be a patch to dbgcnt.def.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42220
--- Comment #31 from dominiq at lps dot ens dot fr 2010-02-18 14:06 ---
> It looks like there should be a patch to dbgcnt.def.
Does this mean that I should remove the line
+ enum debug_counter rnreg;
I have added, and add a line
DEBUG_COUNTER (rnreg)
in dbgcnt.def?
--
http://g
--- Comment #4 from paolo dot carlini at oracle dot com 2010-02-18 14:10
---
Created an attachment (id=19907)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19907&action=view)
Draft patch
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40856
--- Comment #5 from paolo dot carlini at oracle dot com 2010-02-18 14:11
---
Gaby, I just attached a draft patch which essentially does what submitter
requested, adds the two specializations. Shall we do this?
--
paolo dot carlini at oracle dot com changed:
What|Remo
--- Comment #32 from bernds_cb1 at t-online dot de 2010-02-18 14:17 ---
Created an attachment (id=19908)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19908&action=view)
Additional patch on top of the previous one
Sorry about that. Yes, you'll need to add that in dbgcnt.def, or j
--- Comment #1 from paolo dot carlini at oracle dot com 2010-02-18 14:18
---
*** This bug has been marked as a duplicate of 9335 ***
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #5 from paolo dot carlini at oracle dot com 2010-02-18 14:18
---
*** Bug 43113 has been marked as a duplicate of this bug. ***
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #33 from dominiq at lps dot ens dot fr 2010-02-18 14:22 ---
> Sorry about that. Yes, you'll need to add that in dbgcnt.def, or just apply
> this additional patch.
This recompiles most of gcc!-(it will take a couple hours on my poor G5!-).
--
http://gcc.gnu.org/bugzilla
--- Comment #6 from jamborm at gcc dot gnu dot org 2010-02-18 14:53 ---
Subject: Bug 43066
Author: jamborm
Date: Thu Feb 18 14:53:05 2010
New Revision: 156863
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156863
Log:
2010-02-18 Martin Jambor
PR tree-optimization/430
--- Comment #10 from jakub at gcc dot gnu dot org 2010-02-18 15:04 ---
Created an attachment (id=19909)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19909&action=view)
gcc45-pr42233-2.patch
Trunk patch that seems to fix this together with the previous patch.
Going to bootstrap/re
--- Comment #34 from dominiq at lps dot ens dot fr 2010-02-18 15:08 ---
And the winner is N=137!
[karma] f90/bug% gfc -fdbg-cnt=rnreg:137 -m64 -O1 -frename-registers
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/complex_intrinsic_5.f90
dbg_cnt 'rnreg' set to 137
[karma] f90/bug% a.out
[
--- Comment #1 from matz at gcc dot gnu dot org 2010-02-18 15:13 ---
Created an attachment (id=19910)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19910&action=view)
candidate fix
Try this. I don't know how to write guality tests, but I think it has the
effects you look for, at
--- Comment #18 from mmitchel at gcc dot gnu dot org 2010-02-18 15:13
---
Paolo --
I think libsupc++ is just as much a "system library" as libstdc++. It doesn't
have an ISO-standard API, of course, but that's not the point; it's just as
much a part of the system/implementation as libs
--- Comment #19 from paolo dot carlini at oracle dot com 2010-02-18 15:24
---
Ok, it's just an handful of files, after all. If Julian wants to add the
pragmas, the change is pre-approved, otherwise, just let me know when you have
the other issues under control, and I'll do it.
--
h
--- Comment #35 from bernds_cb1 at t-online dot de 2010-02-18 15:32 ---
Okay, great. Could you attach the .rnreg dumps and assembly output for both
values?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42220
--- Comment #36 from dominiq at lps dot ens dot fr 2010-02-18 15:38 ---
> Could you attach the .rnreg dumps
How do I get them?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42220
--- Comment #5 from hjl dot tools at gmail dot com 2010-02-18 15:40 ---
Revision 156792:
http://gcc.gnu.org/ml/gcc-cvs/2010-02/msg00375.html
made ICE disappear. I don't know if it really fixes
the bug or just hides it.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43087
--- Comment #37 from dominiq at lps dot ens dot fr 2010-02-18 15:40 ---
Created an attachment (id=19911)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19911&action=view)
Assembly for -fdbg-cnt=rnreg:137
Command used
gfc -S -fdbg-cnt=rnreg:137 -m64 -O1 -frename-registers
/opt/gcc/
--- Comment #38 from dominiq at lps dot ens dot fr 2010-02-18 15:41 ---
Created an attachment (id=19912)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19912&action=view)
Assembly for -fdbg-cnt=rnreg:138
Command line
gfc -S -fdbg-cnt=rnreg:138 -m64 -O1 -frename-registers
/opt/gcc/
--- Comment #2 from jakub at gcc dot gnu dot org 2010-02-18 15:46 ---
Created an attachment (id=19913)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19913&action=view)
gcc45-pr43077-test.patch
Here is a guality testcase for it. Before the patch there is just lots of
FAILs, with t
--- Comment #39 from bernds_cb1 at t-online dot de 2010-02-18 15:52 ---
(In reply to comment #36)
> > Could you attach the .rnreg dumps
>
> How do I get them?
>
-fdump-rtl-rnreg-details
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42220
--- Comment #40 from dominiq at lps dot ens dot fr 2010-02-18 15:58 ---
Created an attachment (id=19914)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19914&action=view)
.rnreg for -fdbg-cnt=rnreg:137
Command used
fc -fdump-rtl-rnreg-details -fdbg-cnt=rnreg:137 -m64 -O1 -frename-
--- Comment #41 from dominiq at lps dot ens dot fr 2010-02-18 15:59 ---
Created an attachment (id=19915)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19915&action=view)
.rnreg for -fdbg-cnt=rnreg:138
Command line
gfc -fdump-rtl-rnreg-details -fdbg-cnt=rnreg:138 -m64 -O1 -frename
--- Comment #3 from matz at gcc dot gnu dot org 2010-02-18 16:12 ---
At least varb isn't printed in fn3, because dse2 (after prologue_epilogue)
makes the expressions in the debug_insn be (clobber 0). Presumably because
frame elimination rewrites the address-takings, but not the debug in
--- Comment #4 from burnus at gcc dot gnu dot org 2010-02-18 16:19 ---
Another request for this feature:
http://gcc.gnu.org/ml/fortran/2010-02/msg00144.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41209
--- Comment #1 from jason at gcc dot gnu dot org 2010-02-18 16:27 ---
Subject: Bug 43109
Author: jason
Date: Thu Feb 18 16:27:07 2010
New Revision: 156864
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156864
Log:
PR c++/43109
* semantics.c (begin_class_definitio
--- Comment #2 from jason at gcc dot gnu dot org 2010-02-18 16:27 ---
Subject: Bug 43101
Author: jason
Date: Thu Feb 18 16:27:18 2010
New Revision: 156865
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156865
Log:
PR c++/26261
PR c++/43101
* pt.c (tsubst_
--- Comment #13 from jason at gcc dot gnu dot org 2010-02-18 16:27 ---
Subject: Bug 26261
Author: jason
Date: Thu Feb 18 16:27:18 2010
New Revision: 156865
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156865
Log:
PR c++/26261
PR c++/43101
* pt.c (tsubst
--- Comment #3 from jason at gcc dot gnu dot org 2010-02-18 16:28 ---
*** This bug has been marked as a duplicate of 26261 ***
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #14 from jason at gcc dot gnu dot org 2010-02-18 16:28 ---
*** Bug 43101 has been marked as a duplicate of this bug. ***
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26261
--- Comment #15 from jason at gcc dot gnu dot org 2010-02-18 16:28 ---
Fixed for 4.5.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #4 from matz at gcc dot gnu dot org 2010-02-18 16:40 ---
Created an attachment (id=19916)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19916&action=view)
do reg elimination
This will fix some more of the var[bc] references by eliminating
all registers in DEBUG_INSN_P.
--- Comment #5 from matz at gcc dot gnu dot org 2010-02-18 16:50 ---
That leaves 'c' which isn't printed, because the setup of c (a stackslot)
is schedules to be part of the function call setup, the breakpoint is before
that setup, and hence the location of 'c' does point to the correct
--- Comment #31 from jason at gcc dot gnu dot org 2010-02-18 16:52 ---
Performance regression.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Keyw
--- Comment #8 from paolo dot carlini at oracle dot com 2010-02-18 16:56
---
For the record, all the compilers I have at hand, EDG based too, accept this in
the most strict mode. I seriously doubt there is really something to fix here.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jason at gcc dot gnu dot org
|dot org
--- Comment #2 from jason at gcc dot gnu dot org 2010-02-18 17:42 ---
The problem turns out to be that the patch for PR 6196 was wrong;
ext/label[12].C are ill-formed, as both are trying to goto the address of an
array rather than an element.
--
jason at gcc dot gnu dot org changed:
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jason at gcc dot gnu dot org
|dot org
listing of the gfortran program whith the bug
use iso_fortran_env
implicit none
integer,parameter:: mligne = 1,n2 = 10
character(len = n2 + 2):: ligne ! the bug is here : 2 is not necessary
for reading essai.f95 : 10 max character per line
! character(len = n2)::
--- Comment #42 from bernds_cb1 at t-online dot de 2010-02-18 18:13 ---
Created an attachment (id=19917)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19917&action=view)
Another test patch that attempts to fix the problem.
Could you test whether this fixes it?
--
bernds_cb1 at
--- Comment #1 from kargl at gcc dot gnu dot org 2010-02-18 18:15 ---
Please attach the files to the PR. Cut and paste from
bugzilla will mangle the files; particularly, if it is
a CRLF problem.
Also, try removing your iostat usage and see if the
the operating system generates an erro
--- Comment #4 from pinskia at gcc dot gnu dot org 2010-02-18 18:31 ---
(gdb) p debug_gimple_stmt (new_stmt)
# DEBUG p => &s
This was caused by the introduction of debug statements.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #2 from pinskia at gcc dot gnu dot org 2010-02-18 18:36 ---
Happens on x86_64-linux-gnu also even without -ftree-vectorize.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #2 from pinskia at gcc dot gnu dot org 2010-02-18 18:38 ---
Confirmed backtrace:
#0 0x00801625 in link_block (b=0x7f9f943dcaf8, after=0x7f9f943dc138)
at /home/apinski/src/gcc-fsf/local//gcc/gcc/cfg.c:153
#1 0x012b8fb6 in create_bb (h=0x0, e=0x0, after=0x7f9f
--- Comment #3 from pinskia at gcc dot gnu dot org 2010-02-18 18:39 ---
#8 0x0253271e in create_single_exit_edge (region=0x47afff8) at
/home/apinski/src/gcc-fsf/local//gcc/gcc/graphite-scop-detection.c:965
965 forwarder = make_forwarder_block (exit, &sd_region_without_exit
--- Comment #4 from pinskia at gcc dot gnu dot org 2010-02-18 18:42 ---
(In reply to comment #3)
> #8 0x0253271e in create_single_exit_edge (region=0x47afff8) at
> /home/apinski/src/gcc-fsf/local//gcc/gcc/graphite-scop-detection.c:965
> 965 forwarder = make_forwarder_block
--- Comment #43 from dominiq at lps dot ens dot fr 2010-02-18 18:44 ---
The compilation of gcc/regrename.c fails with
...
cc1: warnings being treated as errors
../../gcc-4.5-work/gcc/regrename.c: In function 'build_def_use':
../../gcc-4.5-work/gcc/regrename.c:1113:6: error: array subscr
--- Comment #5 from pinskia at gcc dot gnu dot org 2010-02-18 18:46 ---
Hmm, this code is undefined first but that does not matter. The issue comes
down to return statements not being combined into just one return statement. I
don't know if this is supposed to happen this way or not.
--- Comment #23 from bkoz at redhat dot com 2010-02-18 19:09 ---
Subject: Re: man page errors for generated libstdc++
man pages
> 2010-02-17 09:36 --- So... shall we close this as fixed?
Remaining item is doxygen function markup is not working for man pages.
But this is an upstr
--- Comment #44 from dominiq at lps dot ens dot fr 2010-02-18 19:10 ---
The patch in comment #43 with the fix in comment #44 works for the limited
tests I am able to do right now. I can do a "full" test with a fresh bootstrap
of gcc and fortran, but it will take a full day, so I'ld pref
--- Comment #3 from jason at gcc dot gnu dot org 2010-02-18 19:20 ---
Subject: Bug 43070
Author: jason
Date: Thu Feb 18 19:20:21 2010
New Revision: 156872
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156872
Log:
PR c++/43070
* semantics.c (finish_goto_stmt): Do
--- Comment #20 from manu at gcc dot gnu dot org 2010-02-18 19:44 ---
(In reply to comment #15)
> Subject: Re: warnings about 'mangling of 'va_list' has changed
> in GCC 4.4' not suppressed in sytem headers
>
> manu at gcc dot gnu dot org wrote:
>
> > Why is this a note and not simpl
--- Comment #8 from steven at gcc dot gnu dot org 2010-02-18 19:46 ---
Olga, is this fixed?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41843
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW |WAITING
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41843
--- Comment #21 from mark at codesourcery dot com 2010-02-18 19:47 ---
Subject: Re: warnings about 'mangling of 'va_list' has changed
in GCC 4.4' not suppressed in sytem headers
manu at gcc dot gnu dot org wrote:
> In any case, using diagnostic_report_warnings_p (location) should fix
--- Comment #5 from steven at gcc dot gnu dot org 2010-02-18 19:48 ---
Olga, is this fixed?
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
Stat
--- Comment #6 from jakub at gcc dot gnu dot org 2010-02-18 19:49 ---
I doubt http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43077#c4 applies cleanly to
the current tree, as
http://gcc.gnu.org/viewcvs?view=revision&revision=156693
changed the same spot as the second hunk. Will need to see
--- Comment #32 from jason at gcc dot gnu dot org 2010-02-18 19:58 ---
Subject: Bug 43108
Author: jason
Date: Thu Feb 18 19:58:41 2010
New Revision: 156874
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156874
Log:
PR c++/43108
* typeck.c (cp_build_binary_op): Ad
--- Comment #33 from jason at gcc dot gnu dot org 2010-02-18 19:59 ---
Fixed.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #2 from jason at gcc dot gnu dot org 2010-02-18 20:01 ---
Fixed.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #34 from paolo dot carlini at oracle dot com 2010-02-18 20:18
---
Thanks Jason!
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43108
--- Comment #9 from bangerth at gmail dot com 2010-02-18 20:29 ---
(In reply to comment #8)
> For the record, all the compilers I have at hand, EDG based too, accept this
> in
> the most strict mode. I seriously doubt there is really something to fix here.
That said: if it is unclear t
--- Comment #10 from paolo dot carlini at oracle dot com 2010-02-18 20:36
---
I'm not sure to fully understand, Wolfgang: you mean, we should change that
line in the library instead of dealing with a possible C++ issue here? That
would be easy to do, just tell me the exact form you woul
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jason at gcc dot gnu dot org
|dot org
--- Comment #4 from jason at gcc dot gnu dot org 2010-02-18 20:47 ---
Fixed.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #8 from Zachary_Deretsky at mentor dot com 2010-02-18 20:50
---
With -Wconversion for the assignment to bitfields gcc 4.4.2 gives a
warning, which is impossible to fix.
This BUG (I hope everybody agrees it is a BUG) gives us a lot of
trouble while porting our code (45 devel
--- Comment #11 from bangerth at gmail dot com 2010-02-18 20:53 ---
(In reply to comment #10)
> I'm not sure to fully understand, Wolfgang: you mean, we should change that
> line in the library instead of dealing with a possible C++ issue here? That
> would be easy to do, just tell me th
--- Comment #12 from paolo dot carlini at oracle dot com 2010-02-18 20:55
---
Bah, I can do the change, sure, but if we are not resolving this issue, we are
not making progress on the number of open, long-standing, Bugzilla entries, and
that is *bad*.
--
http://gcc.gnu.org/bugzilla
--- Comment #13 from paolo dot carlini at oracle dot com 2010-02-18 20:57
---
The library issue doesn't exist anymore ;) Thus, let's not be distracted by the
trivial library case, ok?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
--- Comment #9 from olga at il dot ibm dot com 2010-02-18 21:07 ---
(In reply to comment #8)
> Olga, is this fixed?
I think so.
Olga.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41843
--- Comment #24 from paolo dot carlini at oracle dot com 2010-02-18 22:03
---
Richard, can you comment on this issue? Do you think it's currently correct to
have numeric_limits<>:is_modulo == true for all our signed integral types? We
are not making any progress on this issue :(
--
--- Comment #36 from mikestump at comcast dot net 2010-02-18 22:06 ---
I've checked in a slightly updated fix in r156877. Let us know if all the
regressions are fixed now.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43061
--- Comment #6 from rearnsha at gcc dot gnu dot org 2010-02-18 23:09
---
Please supply pre-processed source that will allow a developer to reproduce the
problem.
--
rearnsha at gcc dot gnu dot org changed:
What|Removed |Added
-
1 - 100 of 131 matches
Mail list logo
