This testcase:
#define debug(format, ...) format, ## __VA_ARGS__)
debug(X);
debug(Y, 1, 2);
debug(Y, 1, 2 );
debug(Z, );
debug(W,);
Should preprocess to:
X);
Y, 1, 2);
Y, 1, 2);
Z, );
W, );
not:
X);
Y, 1, 2);
Y, 1, 2);
Z,);
W,);
Z/W should have spaces.
--
Summary: preprocessor
--- Comment #9 from steven at gcc dot gnu dot org 2008-01-29 07:29 ---
Zap.
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #14 from tammer at tammer dot net 2008-01-29 07:03 ---
Hello,
I just tested HEAD on AIX 6.1 and you are right ... the build crashes here:
/opt/freeware/bin/bash ../../gcc-4.3.0/gcc/../move-if-change tmp-attr.h
insn-attr.h
echo timestamp > s-attr
/opt/freeware/src/packages/BU
--- Comment #15 from jv244 at cam dot ac dot uk 2008-01-29 06:31 ---
the comment #10 test case is still broken, and reaching its 3rd anniversary
soon. Just a back trace to the segfault:
1347 cons = cons->next;
(gdb) bt
#0 0x00422cda in find_array_section (expr=0xf7
--- Comment #8 from jason at gcc dot gnu dot org 2008-01-29 05:53 ---
Subject: Bug 35007
Author: jason
Date: Tue Jan 29 05:52:20 2008
New Revision: 131933
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131933
Log:
PR c++/35007
* class.c (build_base_path): Fix !wa
--- Comment #7 from jason at gcc dot gnu dot org 2008-01-29 05:52 ---
Subject: Bug 35007
Author: jason
Date: Tue Jan 29 05:52:13 2008
New Revision: 131932
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131932
Log:
PR c++/35007
* class.c (build_base_path): Fix !wa
--- Comment #6 from jason at gcc dot gnu dot org 2008-01-29 05:51 ---
Subject: Bug 35007
Author: jason
Date: Tue Jan 29 05:50:24 2008
New Revision: 131931
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131931
Log:
PR c++/35007
* class.c (build_base_path): Fix !wa
--- Comment #4 from hp at gcc dot gnu dot org 2008-01-29 05:39 ---
I'm learning; first time around this new df and cfglayout stuff. I haven't
stumbled on anything anywhere actually explicitly saying so, but the "fake"
fallthrough edges are apparently the main part of cfglayout; better t
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2008-01-29 05:17
---
Code in Comment #10 is invalid. I have a fix for that segfault though.
Continuing the hunt. :)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34828
--- Comment #1 from pmarques at grupopie dot com 2008-01-29 04:13 ---
Created an attachment (id=15045)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15045&action=view)
patch to allow target to set double type as unsupported
with this patch, the target can simply add:
set_board_i
--
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
from my original report:
I have a program which uses conditional format tags, similar to this testcase
which is taken from page 187 of Metcalf, Reid and Cohen's Fortran 95/2003
explained:
---
program testcase
integer :: ival(30),key,i
character(30) :: buffer
character(6) :
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2008-01-29 03:33
---
Here is a modified test case that illustrates the problem better. It was just
dumb luck that the test case was passing on some platforms. The following
segfaults on x86-64 with current trunk.
module abuse_mod
The following testcase fails to compile:
template struct outer1
{
typedef int my_value_type; // define a nested datatype
template struct inner1
{
typedef outer1 myboss;
// borrow the boss's datatype << -- don't reduce or change this one:
typedef typename myboss::my
--- Comment #5 from ismail at pardus dot org dot tr 2008-01-29 02:11
---
Yes it works if you revert c++/27177 fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35007
--- Comment #4 from pcarlini at suse dot de 2008-01-29 02:05 ---
This is a reduced snippet (-pedantic). I suspect the problem may have to do
with the fix for c++/27177, let's CC Jason...
struct AffEntry
{
union {
char base[256];
} conds;
};
struct PfxEntry
: public AffEntry
{
--- Comment #3 from dfranke at gcc dot gnu dot org 2008-01-29 01:26 ---
Mine. For patch, see URL above.
--
dfranke at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from roger at eyesopen dot com 2008-01-29 01:12 ---
I'm also seeing this same failure with "make profiledbootstrap" on
x86_64-unknown-linux-gnu. A "make bootstrap" on the same machine completes and
regression tests fine (14 unexpected failures in gcc). I suspect that the
--- Comment #3 from ismail at pardus dot org dot tr 2008-01-29 01:07
---
Compilation line is :
c++ -o affentry.o -c -I../../../../dist/include/system_wrappers -include
../../../../config/gcc_hidden.h -DMOZILLA_INTERNAL_API -D_IMPL_NS_COM
-DEXPORT_XPT_API -DEXPORT_XPTC_API -D_IMPL_NS_CO
sion 4.3.0 20080128 [trunk revision 131899] (Pardus Linux)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35007
--- Comment #1 from ismail at pardus dot org dot tr 2008-01-29 01:04
---
Created an attachment (id=15044)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15044&action=view)
preprocessed file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35007
This is a very latent regression at most introduced under 48 hours. I will
attach preprocessed file.
--
Summary: [4.3 Regression] Firefox fails to build with
affentry.cpp:94: error: ISO C++ forbids subscripting
non-lvalue array
Produc
--- Comment #3 from pinskia at gcc dot gnu dot org 2008-01-29 00:48 ---
Here is a reduced testcase:
typedef unsigned long grub_uint64_t;
typedef grub_uint64_t grub_size_t;
grub_cmdline_get (
unsigned max_len,
int echo_char
)
{
unsigned xpos, ypos, ystart;
grub_size_t lpos, llen;
--- Comment #4 from michael dot meissner at amd dot com 2008-01-29 00:39
---
Created an attachment (id=15043)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15043&action=view)
Proposed patch to fix the problem
The problem is cp/cp-tree.h stores the tree_code in 8 bits, but the tre
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
CC||pinskia at gcc dot gnu dot
|
--- Comment #9 from pinskia at gcc dot gnu dot org 2008-01-29 00:19 ---
With the trunk we get:
return *(data + (long unsigned int) j * 4);
Which is much better. The cast is still needed though.
--
pinskia at gcc dot gnu dot org changed:
What|Removed
--- Comment #2 from tbm at cyrius dot com 2008-01-29 00:18 ---
Created an attachment (id=15042)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15042&action=view)
preprocessed source
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35006
--- Comment #1 from tbm at cyrius dot com 2008-01-29 00:17 ---
Program received signal SIGSEGV, Segmentation fault.
remove_unused_locals () at gcc/tree-ssa-live.c:587
587 var_ann (t)->used = false;
(gdb) where
#0 remove_unused_locals () at gcc/tree-ssa-live.c:587
#1 0x0
With trunk:
(sid)4409:[EMAIL PROTECTED]: ~] /usr/lib/gcc-snapshot/bin/gcc -O3
~/grub2-cmdline.i
/home/tbm/src/grub2-1.95+20080128/normal/cmdline.c: In function
'grub_cmdline_get':
/home/tbm/src/grub2-1.95+20080128/normal/cmdline.c:216: internal compiler
error: Segmentation fault
Pleas
--- Comment #8 from pinskia at gcc dot gnu dot org 2008-01-29 00:17 ---
I think this is basically fixed with the pointer plus work I did.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16187
--- Comment #3 from pinskia at gcc dot gnu dot org 2008-01-29 00:15 ---
I think this is the same as PR 34397 really.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from michael dot meissner at amd dot com 2008-01-29 00:10
---
Created an attachment (id=15041)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15041&action=view)
Traceback for 35005
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35004
The new testcase execute/20071211-1.c assumes 32 bit integers and fails on
machines with 16 bit integers.
--
Summary: New testcase execute/20071211-1.c assumes 32 bit
integers
Product: gcc
Version: 4.3.0
Status: UNCONFIRMED
--- Comment #1 from michael dot meissner at amd dot com 2008-01-29 00:04
---
Created an attachment (id=15040)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15040&action=view)
Preprocessed file from the build of the libstdc++ pre-compiled headers
File is bzip2'ed -9.
--
http:
If I add 4 more tree codes to tree.def, it causes a segmentation violation in
building libstdc++ pre-compiled header files.
Here is the patch to add 4 more tree codes:
--- gcc/tree.def.~1~2007-11-01 11:59:47.0 -0400
+++ gcc/tree.def2008-01-28 16:01:36.0 -0500
@@ -682,6
--- Comment #4 from dgregor at gcc dot gnu dot org 2008-01-28 23:40 ---
There are some problems with the example code. For example, the expression
max does not expand the parameter pack "Params". The compiler should
have warned you about this, both in this case and in the static_assert c
--- Comment #7 from aesok at gcc dot gnu dot org 2008-01-28 22:49 ---
Fixed in HEAD (4.3)
--
aesok at gcc dot gnu dot org changed:
What|Removed |Added
Status|
--- Comment #6 from aesok at gcc dot gnu dot org 2008-01-28 22:40 ---
Subject: Bug 34412
Author: aesok
Date: Mon Jan 28 22:39:11 2008
New Revision: 131923
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131923
Log:
PR target/34412
* config/avr/avr.c (expand_prolog
--- Comment #5 from danglin at gcc dot gnu dot org 2008-01-28 22:19 ---
This was introduced by the addition of generic TLS support.
--
danglin at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #9 from jason at gcc dot gnu dot org 2008-01-28 22:01 ---
changed target milestone.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Target Mile
--- Comment #4 from dgregor at gcc dot gnu dot org 2008-01-28 22:08 ---
This one's mine.
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo
--- Comment #7 from dgregor at gcc dot gnu dot org 2008-01-28 22:05 ---
Darn! Mine again.
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
Statu
--- Comment #1 from dgregor at gcc dot gnu dot org 2008-01-28 21:59 ---
Confirmed. This is mine.
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
As
--- Comment #3 from dgregor at gcc dot gnu dot org 2008-01-28 21:58 ---
Yep, we should be diagnosing but this error but we aren't. I'm on it.
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #8 from dgregor at gcc dot gnu dot org 2008-01-28 21:52 ---
This is fixed on the trunk. Marking it as such...
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #12 from ian at airs dot com 2008-01-28 21:52 ---
Created an attachment (id=15039)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15039&action=view)
Possible patch
Here is a very ugly patch which appears to fix the problem in mainline. I
haven't tested it, though. Any
--- Comment #1 from aldot at gcc dot gnu dot org 2008-01-28 21:47 ---
Confirmed.
Also wrong:
gcc/ipa-struct-reorg.c:sum_counts (d_str str, gcov_type *hotest)
gcc/ipa-struct-reorg.c: if (str->count > *hotest)
gcc/ipa-struct-reorg.c:*hotest = str->count;
gcc/ipa-struct-reorg.c: gcov
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |dgregor at gcc dot gnu dot
|dot org
--- Comment #15 from dgregor at gcc dot gnu dot org 2008-01-28 21:35
---
Thanks everyone; I can reproduce this on i686-pc-linux-gnu. Patch in the works.
--
dgregor at gcc dot gnu dot org changed:
What|Removed |Added
---
Part of original report:
I have recently had the chance to compile some code using -Wconversion and a
loop that may iterate more than 2.2 billion times. There is a compiler warning
of this sort which I cannot resolve:
nell:~/work/bit barnes$ gfortran -Wconversion -o t.x testcase.f
testcase.f:7.1
--- Comment #6 from hubicka at ucw dot cz 2008-01-28 20:51 ---
Subject: Re: [4.3 regression] ICE with -fipa-cp -ffast-math
> No, I mean providing something like cgraph_update_edges_for_call_stmt (tree
> old, tree new); or alternatively cgraph_remove_edge_from_call_stmt () and
> cgraph_
--- Comment #20 from aldot at gcc dot gnu dot org 2008-01-28 20:43 ---
(In reply to comment #19)
> In your patch, why does install-info still need doc and installdirs
> dependencies when BUILD_INFO is not set? If those things still need to
> happen,
> shouldn't they be dependencies of
--- Comment #15 from ian at gcc dot gnu dot org 2008-01-28 19:44 ---
Subject: Bug 33407
Author: ian
Date: Mon Jan 28 19:43:51 2008
New Revision: 131916
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131916
Log:
PR c++/34862
PR c++/33407
* tree-ssa-copyren
--- Comment #11 from ian at gcc dot gnu dot org 2008-01-28 19:44 ---
Subject: Bug 34862
Author: ian
Date: Mon Jan 28 19:43:51 2008
New Revision: 131916
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131916
Log:
PR c++/34862
PR c++/33407
* tree-ssa-copyren
In gcc/params.def there is a message
The threshold ratio between current and hotest structure counts
Unless I'm mistaken, the correct spelling is "hottest".
--
Summary: Incorrect spelling of "hottest"
Product: gcc
Version: 4.3.0
Status: UNCONFIRM
--
tkoenig at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |tkoenig at gcc dot gnu dot
|dot org
--- Comment #9 from tkoenig at gcc dot gnu dot org 2008-01-28 19:07 ---
Fixed on trunk.
Closing (so we're one regression down).
(In reply to comment #7)
> The commit fixed the SHAPE(scalar) problem in the front end.
> a) The following should print "1 0" but it prints "1 -8":
>
> inte
>From PR 34980, comment#7 by Tobias Burnus:
a) The following should print "1 0" but it prints "1 -8":
integer :: i,j, a(10,10),res(2)
j = 1
i = 10
res = shape(a(1:1,i:j:1))
print *, res
res = shape(a(1:1,j:i:-1))
print *, res
end
Not a regression.
--
Summary: shape for negative siz
--- Comment #8 from tkoenig at gcc dot gnu dot org 2008-01-28 19:03 ---
Subject: Bug 34980
Author: tkoenig
Date: Mon Jan 28 19:02:47 2008
New Revision: 131915
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131915
Log:
2008-01-27 Thomas Koenig <[EMAIL PROTECTED]>
PR li
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-01-28 19:02 ---
This is working as designed. Why are you doing #include inside a namespace
anyways?
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from froydnj at gcc dot gnu dot org 2008-01-28 18:32 ---
Subject: Bug 31535
Author: froydnj
Date: Mon Jan 28 18:31:19 2008
New Revision: 131914
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131914
Log:
gcc/
PR 31535
* config/rs6000/rs6000.c (small
When including a system header in a namespace followed by another system header
seems to prevent declarations being recognized in the headers or some such.
Even happens when the headers are both the same.
Example code:
#ifndef APP_H
#define APP_H
namespace win
{
#include
}
#include
#
--- Comment #10 from ian at airs dot com 2008-01-28 18:12 ---
I'm not proposing to remove CHANGE_DYNAMIC_TYPE_EXPR from 4.3 at this point.
We have some experience with it in, and I don't think we should take it out.
I haven't looked at the actual bug here yet, I was responding to your
--- Comment #19 from mmitchel at gcc dot gnu dot org 2008-01-28 18:12
---
In your patch, why does install-info still need doc and installdirs
dependencies when BUILD_INFO is not set? If those things still need to happen,
shouldn't they be dependencies of some other target? Logically,
--- Comment #1 from burnus at gcc dot gnu dot org 2008-01-28 18:09 ---
I think the size is simply calculated as:
ubound[n] + 1 - lbound[n];
which means:
5+1-8 = -2
At some point the variable is regarded as unsigned which means (for 8 byte
variables): 18446744073709551614, which is
--- Comment #18 from aldot at gcc dot gnu dot org 2008-01-28 17:59 ---
Created an attachment (id=15038)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15038&action=view)
use conditional $(BUILD_INFO) variable for prerequs of info and install-info
targets
Use the variable $(BUILD_IN
--- Comment #7 from burnus at gcc dot gnu dot org 2008-01-28 17:39 ---
The commit fixed the SHAPE(scalar) problem in the front end. For the library to
do:
a) The following should print "1 0" but it prints "1 -8":
integer :: i,j, a(10,10),res(2)
j = 1
i = 10
res = shape(a(1:1,i:j:1))
pr
--- Comment #4 from aldot at gcc dot gnu dot org 2008-01-28 17:30 ---
Invalid code should be diagnosed. Since this particular case even has a flag to
accept the invalid code, the hard error is fine IMHO.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34997
--- Comment #6 from burnus at gcc dot gnu dot org 2008-01-28 17:26 ---
Subject: Bug 34980
Author: burnus
Date: Mon Jan 28 17:25:55 2008
New Revision: 131913
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131913
Log:
2008-01-28 Tobias Burnus <[EMAIL PROTECTED]>
PR libfortr
--- Comment #3 from dominiq at lps dot ens dot fr 2008-01-28 17:21 ---
I always wonder if a hard error is really necessary for such cases? would not a
warning be sufficient? I think there is a flag to change warnings to errors if
necessary (-Werror?).
--
http://gcc.gnu.org/bugzilla/
--- Comment #2 from burnus at gcc dot gnu dot org 2008-01-28 17:09 ---
Thanks for the pointer. I think we need in general to update some of the error
message to point to the relevant option. (For the most common, namely
"-fno-range-check" we do so already; for -std=f* related ones we par
--- Comment #4 from jason at gcc dot gnu dot org 2008-01-28 16:34 ---
As explained in 32029, this code is valid. Fixed by the patch for 33959.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #1 from aldot at gcc dot gnu dot org 2008-01-28 16:34 ---
See also
http://gcc.gnu.org/ml/fortran/2007-01/msg00060.html
specifically the "PS ... -fdollar-ok" there.
--
aldot at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #7 from jason at gcc dot gnu dot org 2008-01-28 16:33 ---
The version with "class" is also valid.
14.6p5: A qualified name used as the name in a mem-initializer-id, a
base-specifier, or an elaborated-type-specifier is implicitly assumed to name a
type, without the use of the
--- Comment #14 from jason at gcc dot gnu dot org 2008-01-28 16:28 ---
Fixed in all open branches.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #30 from jason at gcc dot gnu dot org 2008-01-28 16:28 ---
Subject: Bug 27177
Author: jason
Date: Mon Jan 28 16:27:17 2008
New Revision: 131907
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131907
Log:
PR c++/27177
* class.c (build_base_path): Fix pr
--- Comment #13 from jason at gcc dot gnu dot org 2008-01-28 16:28 ---
Subject: Bug 33959
Author: jason
Date: Mon Jan 28 16:27:17 2008
New Revision: 131907
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131907
Log:
PR c++/27177
* class.c (build_base_path): Fix pr
--- Comment #29 from jason at gcc dot gnu dot org 2008-01-28 16:19 ---
Subject: Bug 27177
Author: jason
Date: Mon Jan 28 16:18:56 2008
New Revision: 131905
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131905
Log:
PR c++/27177
* class.c (build_base_path): Fix pr
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Priority|P3 |P4
Target Milestone|--- |4.3.0
http://gcc
--- Comment #6 from jason at gcc dot gnu dot org 2008-01-28 15:38 ---
Not going to bother fixing on other branches.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #1 from eres at il dot ibm dot com 2008-01-28 15:22 ---
Created an attachment (id=15037)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15037&action=view)
the testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34999
I run built-in-setjmp.c testcase (taken from gcc.c-torture/execute dir);
first with -fprofile-generate -freorder-blocks-and-partition
and then with -fprofile-use -freorder-blocks-and-partition
under x86_64 and ppc64-linux with r131883 mainline and I get the ICE:
vn/build/build/spu/gcc/gcc/testsuit
--- Comment #2 from dfranke at gcc dot gnu dot org 2008-01-28 14:45 ---
Confirmed.
Interestingly:
$> cat pr31463.f90
FUNCTION test() RESULT(f)
REAL :: f ! removed DIMENSION statement
END FUNCTION test
$> gfortran-svn -g -Wall -c pr31463.f90
pr31463.f90:1.25
--- Comment #1 from rsandifo at gcc dot gnu dot org 2008-01-28 14:40
---
Testing a patch.
--
rsandifo at gcc dot gnu dot org changed:
What|Removed |Added
Assigne
gcc.c-torture/execute/20040709-1.c fails for -EL -mips16 -O3 with:
20040709-1.c:93: internal compiler error: in insert_save, at caller-save.c:745
This is due to the new subreg liveness checking. We have a bb with:
A: call
...
B: set of (subreg:HI (reg:SI foo))
...
C: use of
This seems to be a common vendor extension. Example:
REAL*4 PLT$C_HOUSTPIX
INTEGER PLT$C_COMMAND
PARAMETER (PLT$B_OPC=0)
common /abc$def/ PLT$C_HOUSTPIX, PLT$C_COMMAND
end
This compiles using NAG f95 (with the warning "Extension: Name contains $
character") and with
--- Comment #5 from dominiq at lps dot ens dot fr 2008-01-28 14:33 ---
The polyhedron test "aermod.f90" fails with:
aermod.f90: In function 'wake_dfsn2':
aermod.f90:37741: error: unrecognizable insn:
(insn 574 3142 575 38 (parallel [
(set (reg:V4SF 188 [ vect_cst_.30783 ])
--
rsandifo at gcc dot gnu dot org changed:
What|Removed |Added
Status|REOPENED|ASSIGNED
Last reconfirmed|2008-01-28 09:16:56 |2008-01-28
--- Comment #3 from rsandifo at gcc dot gnu dot org 2008-01-28 13:43
---
This isn't the same thing as PR34856.
--
rsandifo at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #19 from dorit at gcc dot gnu dot org 2008-01-28 13:20 ---
> Fixed?
In a way, yes. The problem is avoided by generating too conservative code.
AFAIU, a better solution may be expected in 4.4 from the stack alignment
branch. In any case this segfault PR can be closed, and ins
test.h:
struct Base
{
virtual int getX() const=0;
};
template class Derived1: public Base
{
protected:
static const int X;
public:
virtual int getX() const{ return X; }
};
template class Derived2 : public Derived1 > {};
test.cpp:
#inclu
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-01-28 12:39 ---
No, I mean providing something like cgraph_update_edges_for_call_stmt (tree
old, tree new); or alternatively cgraph_remove_edge_from_call_stmt () and
cgraph_add_edge_from_call_stmt () and call those two unconditional
--- Comment #4 from jakub at gcc dot gnu dot org 2008-01-28 12:34 ---
Not folding CALL_EXPRs would introduce a regression on g++.dg/opt/devirt1.C,
after all fold_marked_statements has been added exactly to fix that regression:
http://gcc.gnu.org/ml/gcc-patches/2007-01/msg00165.html
Rega
--- Comment #9 from dominiq at lps dot ens dot fr 2008-01-28 12:25 ---
My tests have not been as exhaustive as the Tobias' ones, but the patch worked
as advertised without regression on ppc/intel Darwin9, 32 and 64 bit modes.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34975
--- Comment #26 from ubizjak at gmail dot com 2008-01-28 12:04 ---
*** Bug 34995 has been marked as a duplicate of this bug. ***
--
ubizjak at gmail dot com changed:
What|Removed |Added
--
--- Comment #2 from ubizjak at gmail dot com 2008-01-28 12:04 ---
*** This bug has been marked as a duplicate of 34856 ***
--
ubizjak at gmail dot com changed:
What|Removed |Added
--
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-01-28 11:11 ---
I suppose a simpler fix for 4.3 would be to not fold call stmts here.
Otherwise
moving the cgraph adjustment to a helper function in cgraph.c would look
nicer.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=349
--- Comment #8 from burnus at gcc dot gnu dot org 2008-01-28 11:09 ---
> Created an attachment (id=15033)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15033&action=view) [edit]
> A patch for this regression
> I have just put this on to bootstrap and regtest whilst I sleep. Since
--- Comment #10 from manu at gcc dot gnu dot org 2008-01-28 10:51 ---
(In reply to comment #8)
> (In reply to comment #6)
> > But shouldn't -Wno-strict-aliasing suppress the warning?
>
> Yes if you actually have -Wno-strict-aliasing after -Wall.
Andrew, note that this is not longer tru
--- Comment #9 from manu at gcc dot gnu dot org 2008-01-28 10:45 ---
Christoph, we need a testcase to be able to reproduce the bug, otherwise, we
cannot fix it.
--
manu at gcc dot gnu dot org changed:
What|Removed |Added
---
1 - 100 of 114 matches
Mail list logo
