--- Comment #16 from ebotcazou at gcc dot gnu dot org 2009-09-10 06:52
---
> As mentioned by you the precompiled version of GCC on solaris 10 that we are
> using is not the official compiler. So how do i fullfill this prerequisite
Try with Sun Studio or with the genuine GCC: http://ww
--- Comment #15 from vijay dot x dot jain at jpmchase dot com 2009-09-10
06:02 ---
Please find attached stack trace of the segmentation fault
GNU C (GCC) version 4.3.2 (sparc-sun-solaris2.10)
compiled by GNU C version 4.3.2 (20090604) (gccfss), GMP version 4.3.1,
MPFR version
--- Comment #4 from howarth at nitro dot med dot uc dot edu 2009-09-10
03:52 ---
Attached files were generated under r151584 with the command...
/sw/src/fink.build/gcc45-4.4.999-20090909/darwin_objdir/gcc/testsuite/g++/../../g++
-B/sw/src/fink.build/gcc45-4.4.999-20090909
--- Comment #3 from howarth at nitro dot med dot uc dot edu 2009-09-10
03:51 ---
Created an attachment (id=18560)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18560&action=view)
gcda file for g++.dg/tree-prof/partition1.C compilation -g -fprofile-use
--
http://gcc.gnu.org/bu
--- Comment #2 from howarth at nitro dot med dot uc dot edu 2009-09-10
03:50 ---
Created an attachment (id=18559)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18559&action=view)
assembly file for g++.dg/tree-prof/partition1.C compilation -g -fprofile-use
--
http://gcc.gnu.or
--- Comment #1 from howarth at nitro dot med dot uc dot edu 2009-09-10
03:49 ---
Created an attachment (id=18558)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18558&action=view)
preprocessed source for g++.dg/tree-prof/partition1.C compilation -g
-fprofile-use
--
http://gcc.
--- Comment #4 from kargl at gcc dot gnu dot org 2009-09-10 02:43 ---
(In reply to comment #2)
> (In reply to comment #1)
> > Thus the question is: Why is the last expr == NULL and not EXPR_VARIABLE of
> > flavour FL_PARAMETER?
>
> gfc_match_rvalue replaces parameters with their values:
--- Comment #3 from hjl dot tools at gmail dot com 2009-09-10 01:10 ---
It was introduced between revision 151561 and revision 151575.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41324
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2009-09-10 01:08
---
Features, features, features, always features... :)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41298
--- Comment #2 from kargl at gcc dot gnu dot org 2009-09-09 23:52 ---
Problem is also seen of i386-*-freebsd
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41324
--- Comment #1 from kargl at gcc dot gnu dot org 2009-09-09 23:50 ---
*** Bug 41326 has been marked as a duplicate of this bug. ***
--
kargl at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #1 from kargl at gcc dot gnu dot org 2009-09-09 23:50 ---
*** This bug has been marked as a duplicate of 41324 ***
--
kargl at gcc dot gnu dot org changed:
What|Removed |Added
--
gmake[3]: Leaving directory `/usr/nabil-files/pfeifer/OBJ-0909-2216'
Comparing stages 2 and 3
warning: gcc/cc1-checksum.o differs
warning: gcc/cc1plus-checksum.o differs
--- Comment #9 from paolo dot carlini at oracle dot com 2009-09-09 23:34
---
Fixed in mainline.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #8 from paolo at gcc dot gnu dot org 2009-09-09 23:34 ---
Subject: Bug 28293
Author: paolo
Date: Wed Sep 9 23:33:38 2009
New Revision: 151581
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151581
Log:
/cp
2009-09-09 Paolo Carlini
PR c++/28293
* d
--- Comment #7 from paolo at gcc dot gnu dot org 2009-09-09 23:32 ---
Subject: Bug 28293
Author: paolo
Date: Wed Sep 9 23:31:47 2009
New Revision: 151580
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151580
Log:
/cp
2009-09-09 Paolo Carlini
PR c++/28293
* d
--- Comment #7 from rsc at swtch dot com 2009-09-09 23:30 ---
There is a difference between warnings about things that might
be wrong in your code and warnings about things that are definitely wrong.
This is one of the latter, and it is a shame that it's not already an error
by default.
--- Comment #1 from meissner at gcc dot gnu dot org 2009-09-09 22:57
---
Created an attachment (id=18557)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18557&action=view)
Patch to require vector addresses for GPRs to be a single register
--
http://gcc.gnu.org/bugzilla/show_bu
On trunk rev 151524 (+ Mikael patch) there is only one ACATS fail
,.,. C37105A ACATS 2.5 09-09-09 18:49:01
C37105A RECORDS WITH ONLY DISCRIMINANTS.
* C37105A DISCRIMINANT-ONLY RECORDS DON'T WORK.
C37105A FAILED .
--
Summary: ACATS c37105a wro
--- Comment #2 from rth at gcc dot gnu dot org 2009-09-09 22:35 ---
Register selection was slightly better in 4.3.5; mainline has a few
extra move instructions.
And for the record, "size test.o" avoids the need for the objcopy:
textdata bss dec hex filename
66
--- Comment #1 from rth at gcc dot gnu dot org 2009-09-09 22:14 ---
This is almost certainly a bug in the m68k backend in that it
doesn't check for the difference in return registers. Compare
it's m68k_ok_for_sibcall_p function vs the i386 routine.
--
rth at gcc dot gnu dot org chan
--- Comment #6 from jason at redhat dot com 2009-09-09 21:52 ---
Subject: Re: ICE on invalid typedef
On 09/09/2009 12:25 PM, paolo dot carlini at oracle dot com wrote:
> Jason, any chance you can have a look to the old patch of mine for this PR?
The patch is OK.
Jason
--
http://
--- Comment #15 from rth at gcc dot gnu dot org 2009-09-09 21:28 ---
(In reply to comment #13)
> Could this chain perhaps be moved to %edx?
No, regparm(3) uses all of %eax, %edx, %ecx. There are no other
available call-clobbered registers.
Incidentally, fastcall functions are currentl
On Linux/ia32, revision 151575 gave:
Comparing stages 2 and 3
warning: gcc/cc1plus-checksum.o differs
warning: gcc/cc1obj-checksum.o differs
warning: gcc/cc1-checksum.o differs
Bootstrap comparison failure!
gcc/gcov.o differs
gcc/gcov-dump.o differs
gcc/coverage.o differs
make[5]: *** [compare] Er
--- Comment #14 from rth at gcc dot gnu dot org 2009-09-09 21:23 ---
Not working on it yet, but I think it should be possible to have the
static chain pushed to the stack by the trampoline. Direct calls to
the nested function would have to pass the static chain in a call
saved register.
I suggest following code (or equivalent) should be added to immintrin.h (if
SSE2 enabled).
extern __inline unsigned
_mm_extract_epu16 (__m128i const __A, int const __N)
{
unsigned __r; __asm__ ("pextrw\t%2,%1,%0" : "=g,m"(__r) : "x,x"(__A),
"i,i"(__N)); return __r;
}
This behaves like _mm_extra
--- Comment #3 from joseph at codesourcery dot com 2009-09-09 20:52 ---
Subject: Re: New: [4.5 Regression] Failed to bootstrap
On Wed, 9 Sep 2009, hjl dot tools at gmail dot com wrote:
> We aren't consistent where to report gcc bugs:
>
> [...@gnu-31 src-trunk]$ grep ttp://gcc.gnu.or
--- Comment #2 from bonzini at gnu dot org 2009-09-09 20:50 ---
has already been reverted.
--
bonzini at gnu dot org changed:
What|Removed |Added
Status|UNCON
--- Comment #2 from mikael at gcc dot gnu dot org 2009-09-09 20:41 ---
(In reply to comment #1)
> Thus the question is: Why is the last expr == NULL and not EXPR_VARIABLE of
> flavour FL_PARAMETER?
gfc_match_rvalue replaces parameters with their values:
case FL_PARAMETER:
/* A
--- Comment #5 from pthaugen at gcc dot gnu dot org 2009-09-09 20:23
---
Didn't mean to change the component when CC'ing myself, changing back.
--
pthaugen at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #9 from pault at gcc dot gnu dot org 2009-09-09 20:07 ---
Fixed on trunk
Thanks richi!
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
On Linux/ia32, revision 151568 gave:
cc1: warnings being treated as errors
In file included from ../libdecnumber/gstdint.h:4:0,
from ../../src-trunk/gcc/../libdecnumber/decContext.h:54,
from ../../src-trunk/gcc/../libdecnumber/decNumber.h:37,
from
--- Comment #8 from pault at gcc dot gnu dot org 2009-09-09 20:04 ---
Subject: Bug 41297
Author: pault
Date: Wed Sep 9 20:03:49 2009
New Revision: 151576
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151576
Log:
2009-09-09 Richard Guenther
PR fortran/41297
--- Comment #1 from hjl dot tools at gmail dot com 2009-09-09 20:03 ---
This may be caused by revision 151567:
http://gcc.gnu.org/ml/gcc-cvs/2009-09/msg00314.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
-
--- Comment #6 from domob at gcc dot gnu dot org 2009-09-09 19:27 ---
(In reply to comment #5)
> OK. It is definitely Daniel's r151140 that has introduced the regression.
> Now
> to try to understand why :-)
If you've no luck with that, I should hopefully find some time over the week
--- Comment #5 from uros at gcc dot gnu dot org 2009-09-09 19:25 ---
Subject: Bug 39779
Author: uros
Date: Wed Sep 9 19:25:31 2009
New Revision: 151573
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151573
Log:
PR rtl-optimization/39779
* expr.c (convert_modes):
--- Comment #3 from ktietz at gcc dot gnu dot org 2009-09-09 19:10 ---
Applied to 4.4.x branch at revision 151571, and to trunk at revision 151570.
--
ktietz at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #5 from pault at gcc dot gnu dot org 2009-09-09 19:06 ---
(In reply to comment #4)
> Wierd! I think that this will require Daniel and Janus to go back over their
> patches in the period concerned or to identify the specific revision. I will
> try to do that tonight.
OK.
--- Comment #2 from paolo dot carlini at oracle dot com 2009-09-09 18:53
---
Current mainline (151568) instead just accepts it. Is it now an accept-invalid?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36993
--- Comment #9 from aoliva at gcc dot gnu dot org 2009-09-09 18:42 ---
(In reply to comment #3)
> Hmm, dropping the stmt looks like it would be a "hack". Alex - if I just set
> flag_var_tracking_assignments to 1 if I encounter a GIMPLE_DEBUG
> is there sth else that I need to do to "en
--- Comment #2 from dje at gcc dot gnu dot org 2009-09-09 17:57 ---
Fixed on mainline. Patch needs to be backported.
--
dje at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #9 from jakub at gcc dot gnu dot org 2009-09-09 17:49 ---
See http://gcc.gnu.org/ml/gcc-patches/2009-09/msg00592.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40677
--- Comment #8 from dje at gcc dot gnu dot org 2009-09-09 16:59 ---
Kumar,
If you want this fixed then you or Edmar or someone at Freescale should fix the
patch referenced in comment #6.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40677
--- Comment #14 from jamborm at gcc dot gnu dot org 2009-09-09 16:50
---
Subject: Bug 41089
Author: jamborm
Date: Wed Sep 9 16:50:15 2009
New Revision: 151566
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151566
Log:
2009-09-09 Martin Jambor
PR tree-optimization/4
--- Comment #1 from aran at 100acres dot us 2009-09-09 16:40 ---
Created an attachment (id=18556)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18556&action=view)
Adds __amd64__ to conditional code
This includes asm("finit") for amd64 targets on NetBSD
--
http://gcc.gnu.org/b
The function __gnat_init_float doesn't call asm("finit") when compiled on
NetBSD for x86_64. It looks like Win32, Interix, emx, Lynx, FreeBSD, and
OpenBDS have the same problem, but this is unconfirmed.
--
Summary: Ada runtime not initializing fpu (finit)
Product: gcc
--
ramana at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last reconfir
--- Comment #5 from paolo dot carlini at oracle dot com 2009-09-09 16:25
---
Jason, any chance you can have a look to the old patch of mine for this PR? I
have regtested again a slightly tweaked version of the original one. Note, the
issue is rather annoying to the users, because curren
--- Comment #4 from paolo dot carlini at oracle dot com 2009-09-09 16:23
---
Created an attachment (id=18555)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18555&action=view)
Slightly tweaked (only the testcase) patch
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28293
--- Comment #13 from ubizjak at gmail dot com 2009-09-09 16:23 ---
(In reply to comment #12)
> My attempts at cross-compiling these testcases seem to indicate that
> no early SRA happens in gcc.dg/tree-ssa/stdarg-2.c (although ap gets
> scalarized by late SRA in f15 but that is too late
--- Comment #12 from jamborm at gcc dot gnu dot org 2009-09-09 16:05
---
(In reply to comment #11)
> (In reply to comment #10)
>
> > Thus I am now bootstrapping and testing the following patch on
> > x86_64-linux. Uros, can you please test it on Alpha? Thanks.
>
> This patch fixes g
--- Comment #5 from matz at gcc dot gnu dot org 2009-09-09 15:46 ---
Oh, maybe an explanation: what is expected is that such decls (externs) either
have no context set (global) or have the current function as context. For
inlining (cloning/versioning all the same) this needs to be done
--- Comment #4 from matz at gcc dot gnu dot org 2009-09-09 15:41 ---
Even shorter:
struct ErrmsgWindow
{
virtual ~ErrmsgWindow()
{
extern int _switch_mode_errorstr;
_switch_mode_errorstr = 42;
}
};
void ShowErrorMessage(void)
{
ErrmsgWindow w;
}
The problem app
--- Comment #5 from paolo dot carlini at oracle dot com 2009-09-09 15:37
---
In fact, with checking disabled no ICE or any other user visible anomalous
behavior, I'm not going to spend time on this.
--
paolo dot carlini at oracle dot com changed:
What|Removed
--- Comment #21 from paolo dot carlini at oracle dot com 2009-09-09 15:25
---
Ok, thanks. I'm assigning the bug to you then.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--
--- Comment #4 from bonzini at gnu dot org 2009-09-09 15:11 ---
I don't see the reload failure anymore.
The code is:
push{r4, lr}
mov r4, r0
mov r1, #0
bl __gesf2
cmp r0, #0
it lt
movlt r4, #0
m
--- Comment #24 from rguenth at gcc dot gnu dot org 2009-09-09 15:05
---
Fixed on trunk. Let's wait a bit to watch for fallout before backporting it.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #23 from rguenth at gcc dot gnu dot org 2009-09-09 15:05
---
Subject: Bug 41101
Author: rguenth
Date: Wed Sep 9 15:04:27 2009
New Revision: 151561
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151561
Log:
2009-09-09 Richard Guenther
PR tree-optimizatio
--- Comment #7 from florian at openwrt dot org 2009-09-09 15:04 ---
Ping ? Anything else I should provide ? The bug is still present with gcc-4.4.1
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40641
--- Comment #2 from rguenth at gcc dot gnu dot org 2009-09-09 15:04 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #1 from rguenth at gcc dot gnu dot org 2009-09-09 15:01 ---
What is now needed is that forwprop takes
a_12 = (int[0:D.2718] *) p_11(D);
q_13 = (int *) a_12;
D.2724_14 = q_13 + 4;
D.2723_15 = *D.2724_14;
and should propagate (int *) a_12 via q_13 + 4 into the derefer
We no longer propagate &(*a)[0] into the dereference in
int f(int *p, int n)
{
int (*a)[n] = (int (*)[n])p;
int *q = &(*a)[0];
return q[1];
}
because 1) the C frontend decomposes &(*a)[0] into address arithmetic and
2) because of the fix for PR41317 we no longer fold (int *)a to &(*a)[0].
--- Comment #9 from nospamname at web dot de 2009-09-09 14:54 ---
ratecontrol.c line 624 cause crash in my source.assert is done by this
command(should be fbngt)
FBNLT _ff_rate_estimate_qscale+$716 ;118F015C
q= get_qscale(s, rce, rate_factor, picture_number);
printf("%f\n",q); // (is
--- Comment #20 from 3dw4rd at verizon dot net 2009-09-09 14:46 ---
(In reply to comment #19)
> Ed, please let me know ASAP if you are going to work on this, otherwise I'll
> do
> it. Thanks.
>
I can do this.
FWIW, I thought I had sort in the base class st some point. I'll look in m
--- Comment #11 from ubizjak at gmail dot com 2009-09-09 14:44 ---
(In reply to comment #10)
> Thus I am now bootstrapping and testing the following patch on
> x86_64-linux. Uros, can you please test it on Alpha? Thanks.
This patch fixes gcc.c-torture/execute/stdarg-4.c and
gcc.dg/tr
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last re
--- Comment #19 from paolo dot carlini at oracle dot com 2009-09-09 14:37
---
Ed, please let me know ASAP if you are going to work on this, otherwise I'll do
it. Thanks.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41316
--- Comment #4 from rguenth at gcc dot gnu dot org 2009-09-09 14:37 ---
Subject: Bug 41317
Author: rguenth
Date: Wed Sep 9 14:35:51 2009
New Revision: 151559
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151559
Log:
2009-09-09 Richard Guenther
PR middle-end/41317
--- Comment #3 from rguenth at gcc dot gnu dot org 2009-09-09 14:36 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #18 from paolo dot carlini at oracle dot com 2009-09-09 14:32
---
Fine, let's go with that.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41316
--- Comment #17 from rguenth at gcc dot gnu dot org 2009-09-09 14:31
---
As I said the proper fix would be to move _M_sort_after to _Fwd_list_node_base
to be able to simply do
void
sort()
{
this->_M_impl._M_head._M_sort_after(std::less<_Tp>());
}
instea
--- Comment #2 from paolo dot carlini at oracle dot com 2009-09-09 14:09
---
Your code is broken. You want *(ps + 1) == 'z'.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--
--- Comment #14 from ebotcazou at gcc dot gnu dot org 2009-09-09 14:08
---
> I ran the same command on command line and got the same error
> conftest.c:1: internal compiler error: Segmentation Fault
> Please submit a full bug report,
> with preprocessed source if appropriate.
> See
--- Comment #1 from lxcc dot it dot gg at email dot it 2009-09-09 14:03
---
Created an attachment (id=18554)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18554&action=view)
source, object, binary and log
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41319
The condition
if ((1+*ps)=='z') .
not have the same behavior of
if ((1+*ps)=='b') .
In the very simple attached file, you can reproduce it.
for to reproduce, you will change 2 rows:
1) row 38 with row 37
2) row 30 with row 29
There is also a log file, so you can see how i compile it.
Ot
--- Comment #16 from paolo dot carlini at oracle dot com 2009-09-09 13:58
---
To be clear: I don't think we want to carry forward ugly, totally brittle,
solutions only for the sake of the C++0x forward_list: this is experimental
stuff, and there are no exported symbols involved.
We guy
--- Comment #2 from ktietz at gcc dot gnu dot org 2009-09-09 13:43 ---
Passed regression tests. Sent patch to ML at
http://gcc.gnu.org/ml/gcc-patches/2009-09/msg00593.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41315
--- Comment #13 from vijay dot x dot jain at jpmchase dot com 2009-09-09
13:39 ---
Created an attachment (id=18553)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18553&action=view)
where gcc4.3.2 gnu compile is failing
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41304
--- Comment #12 from vijay dot x dot jain at jpmchase dot com 2009-09-09
13:38 ---
Created an attachment (id=18552)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18552&action=view)
Solaris 8 config.log where gcc4.3.2 compiles sucessfully
--
http://gcc.gnu.org/bugzilla/show_bu
--- Comment #15 from rguenther at suse dot de 2009-09-09 13:37 ---
Subject: Re: forward_list::sort violates strict aliasing
rules
On Wed, 9 Sep 2009, paolo dot carlini at oracle dot com wrote:
> --- Comment #14 from paolo dot carlini at oracle dot com 2009-09-09
> 13:18 ---
--- Comment #1 from paolo dot carlini at oracle dot com 2009-09-09 13:28
---
Your code is clearly invalid: str points to "test", but that memory itself is
read-only, roughly speaking. More correctly, the Standard says that the effect
of attempting to modify a string literal is undefined
--- Comment #7 from rguenth at gcc dot gnu dot org 2009-09-09 13:28 ---
This is because we now emit thunks for the (unused) destructor. The following
seems to work for both testcases, testing is pending
Index: cgraphunit.c
===
I have stumbled upon a problem that, I believe, can be attributed to a bug in
GCC. The following source code can be used to reproduce the problem:
int foo(void)
{
char * str = "test";
str[1] = 'a';
return 0;
}
int main() {
foo();
return 0;
}
On a Solaris 10 system, I can compile a
--- Comment #14 from paolo dot carlini at oracle dot com 2009-09-09 13:18
---
(In reply to comment #12)
> I need a reinterpret_cast because a regular static_cast would be an
> access as _Fwd_list_node again - at least the FE (or maybe it is
> fold after all) produces &this->D.1234 in t
--- Comment #11 from vijay dot x dot jain at jpmchase dot com 2009-09-09
13:11 ---
Following is the command on solaris 8 which compiles sucessfully
/home/odyssey/f065093/gcc8/gcc8objdir/./gcc/xgcc -v
-B/home/odyssey/f065093/gcc8/gcc8objdir/./gcc/
-B/home/odyssey/f065093/gcc8/gccinstall
--- Comment #6 from reichelt at gcc dot gnu dot org 2009-09-09 13:07
---
Well, now I can't link the following program anymore:
===
struct A
{
virtual ~A();
};
struct B : virtual A
{
virtual ~B() {}
};
int main()
{
return 0;
}
--- Comment #12 from howarth at nitro dot med dot uc dot edu 2009-09-09
13:02 ---
Patch posted at http://gcc.gnu.org/ml/gcc-patches/2009-09/msg00569.html.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41288
--- Comment #13 from rguenth at gcc dot gnu dot org 2009-09-09 12:25
---
I have a fix for PR41317. I'm also pretty sure I have seen this pattern in
other container classes ...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41316
--- Comment #2 from rguenth at gcc dot gnu dot org 2009-09-09 12:16 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #12 from rguenther at suse dot de 2009-09-09 11:44 ---
Subject: Re: forward_list::sort violates strict aliasing
rules
On Wed, 9 Sep 2009, paolo dot carlini at oracle dot com wrote:
> --- Comment #10 from paolo dot carlini at oracle dot com 2009-09-09
> 11:36 ---
--- Comment #1 from rguenth at gcc dot gnu dot org 2009-09-09 11:42 ---
4.4 rightfully complains:
t.i: In function main:
t.i:16: warning: dereferencing pointer a.0 does break strict-aliasing rules
t.i:23: note: initialized from here
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?i
--- Comment #11 from paolo dot carlini at oracle dot com 2009-09-09 11:41
---
Oops, restored the dependency.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--
--- Comment #10 from paolo dot carlini at oracle dot com 2009-09-09 11:36
---
Ah, so, without moving _M_sort_after itself. By the way, I'm not sure to
understand why you need an ugly reinterpret cast, aren't you just going from
pointer to derived to pointer to base? I see the issue is t
extern void abort (void);
struct A
{
int i;
};
struct B
{
struct A a;
int j;
};
static void
foo (struct B *p)
{
((struct A *)p)->i = 1;
}
int main()
{
struct A a;
a.i = 0;
foo ((struct B *)&a);
if (a.i != 1)
abort ();
return 0;
}
Folding (struct A *)p to &p->a causes us t
--- Comment #10 from vijay dot x dot jain at jpmchase dot com 2009-09-09
11:35 ---
I have got the file
| /* confdefs.h. */
|
| #define PACKAGE_NAME "GNU C Runtime Library"
| #define PACKAGE_TARNAME "libgcc"
| #define PACKAGE_VERSION "1.0"
| #define PACKAGE_STRING "GNU C Runtime Librar
--- Comment #9 from rguenther at suse dot de 2009-09-09 11:30 ---
Subject: Re: forward_list::sort violates strict aliasing
rules
On Wed, 9 Sep 2009, paolo dot carlini at oracle dot com wrote:
> --- Comment #8 from paolo dot carlini at oracle dot com 2009-09-09 11:23
> ---
>
--- Comment #8 from paolo dot carlini at oracle dot com 2009-09-09 11:23
---
if I understand correctly what we would change, I'm not sure to like this
casting from _Fwd_list_node_base* to _Fwd_list_node* inside
_Fwd_list_node_base...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4
--- Comment #7 from paolo dot carlini at oracle dot com 2009-09-09 11:17
---
in the member moved to the base class, right? Sure, the ABI is fine.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41316
--- Comment #6 from rguenth at gcc dot gnu dot org 2009-09-09 11:13 ---
The trivial change I'd suggest (and that hasn't the chance to break the ABI)
would be to cast all this->_M_next accesses properly like ((_Base
*)this)->_M_next
Introducing a _Base typedef to _Fwd_list_node.
--
h
--- Comment #3 from ramana at gcc dot gnu dot org 2009-09-09 11:08 ---
There isn't a reload failure with trunk as of yesterday.
Paolo can you explain this further ?
--
ramana at gcc dot gnu dot org changed:
What|Removed |Added
--
1 - 100 of 121 matches
Mail list logo