--- Comment #8 from pinskia at gcc dot gnu dot org 2006-10-23 06:48 ---
Note this looks like a latent bug.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29443
--- Comment #7 from pinskia at gcc dot gnu dot org 2006-10-23 06:47 ---
Reduced testcase:
struct string
{
struct _Rep{}; static _Rep _S_empty_rep_storage[];
void *_M_rep () const{} void _M_destroy () throw ();
~string ()
{
_Rep *a = (_Rep *)((void *)(&_S_empty_re
--- Comment #6 from pinskia at gcc dot gnu dot org 2006-10-23 06:39 ---
(gdb) p debug_rtx(x)
(unspec:DI [
(symbol_ref:DI ("_ZN6string20_S_empty_rep_storageE") [flags 0x40]
)
] 2)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29443
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2006-10-23 06:19
---
Created an attachment (id=12477)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12477&action=view)
Example patch
I don't know if it's giving correct results in all cases, or if it works even
on platforms wit
Background: This problem was found by running gnu mp's "make check" script
after an successfull "make". GNU MP home page:
http://www.swox.com/gmp/
Problem discovered while "make checking" ... of programs from gmp-4.2.1.tar.bz2
$ /usr/local/bin/gcc -v
Using built-in specs.
Tar
--- Comment #5 from pinskia at gcc dot gnu dot org 2006-10-23 04:15 ---
Sorry, not TLS but instead some PIC code. This is a target specific issue.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #4 from pinskia at gcc dot gnu dot org 2006-10-23 04:02 ---
This is TLS that is being ICEd on so I doubt this is a regression, except for
libstdc++ using TLS now.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29443
--- Comment #13 from bangerth at math dot tamu dot edu 2006-10-23 04:01
---
Subject: Re: [4.1 Regression] Internal compiler error on
boost mpl test/apply.cpp
So maybe the question should be a different one: if it is difficult and/or
potentially risky to backport the patch mentioned,
--- Comment #12 from fang at csl dot cornell dot edu 2006-10-23 03:34
---
Subject: Re: [4.1 Regression] Internal compiler error on
boost mpl test/apply.cpp
> There were PRs that showed that this extension (for those who don't
> remember: we allowed to match templates with trailing de
--- Comment #11 from bangerth at math dot tamu dot edu 2006-10-23 03:18
---
Subject: Re: [4.1 Regression] Internal compiler error on
boost mpl test/apply.cpp
> Technically, I think it would be feasible to backport the patch. However,
> because that patch eliminates a GNU extension
--- Comment #5 from pinskia at gcc dot gnu dot org 2006-10-23 02:28 ---
This is also fixed at -O2 -fsee so SEE also fixes the problem.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29083
--- Comment #4 from pinskia at gcc dot gnu dot org 2006-10-23 02:16 ---
Yes this is fixed on the dataflow branch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29083
--- Comment #3 from pinskia at gcc dot gnu dot org 2006-10-23 00:56 ---
Related to PR 29556.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
BugsThisDepen
--- Comment #1 from pinskia at gcc dot gnu dot org 2006-10-23 00:48 ---
Confirmed. Related to:
http://gcc.gnu.org/ml/gcc-patches/2006-10/msg00945.html
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #3 from pinskia at gcc dot gnu dot org 2006-10-22 23:55 ---
Confirmed, what happens is that combine combines:
(insn 18 16 19 3 (set (reg/v/f:SI 123 [ node ])
(plus:SI (reg/v/f:SI 123 [ node ])
(const_int -4 [0xfffc]))) 79 {*addsi3_internal1} (n
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2006-10-22 23:14
---
I've been thinking a bit about this. It's a common case, and it would probably
be worth optimizing it.
We could detect in iresolve.c (gfc_resolve_matmul) that one (or both) of the
arguments to MATMUL is a call to
--- Comment #4 from pinskia at gcc dot gnu dot org 2006-10-22 23:13 ---
(In reply to comment #2)
> This is another example of premature tree optimization, and it should be
> addressed
> in the reworking of fold itself. I don't think this should be attempted at
> stage 3
First off, fo
"using namespace std" does not issue an error or warning when the namespace std
is not declared. Using other namespaces that are not declared issue a warning.
Expects:
"using namespace std;" to issue the error
error: std is not a namespace-name.
when namespace std is not defined.
System:
2.6.
--- Comment #13 from laurent at guerby dot net 2006-10-22 21:55 ---
Indeed :)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29543
I want to build a cross compiler for fr30 architecture (including c++) and
configure in a separate build dir as follows:
../gcc-4.1.1/configure --prefix=/usr/cross --target=fr30-unknown-elf
--with-gnu-as --with-gnu-ld --enable-languages=c --disable-libssp
Then is make and make install, which runs
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2006-10-22 21:01
---
Having thought about it some more, MODULO should be implemented using
fmod{f,,l} and MOD should use remainder{f,,l}. See how BUILT_IN_POWL is used,
for example, to emit the same kind of code in gfc_conv_intrinsic_
--- Comment #8 from aldot at gcc dot gnu dot org 2006-10-22 21:08 ---
Created an attachment (id=12476)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12476&action=view)
untested proposed fix
I hope to test this patch soon.
Note that it adds a warning about blank common in block da
--- Comment #11 from laurent at guerby dot net 2006-10-22 20:55 ---
My point is that the subprogram I wrote is probably the way most Ada
programmers will write when they want to multiply matrices (if not using
BLAS/Lapack).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29543
--- Comment #1 from rguenth at gcc dot gnu dot org 2006-10-22 20:47 ---
In flag ? (unsigned)__x : __x , __x is promoted to unsigned and then promoted
to long, so the code is different than bar.
--
rguenth at gcc dot gnu dot org changed:
What|Removed
--- Comment #5 from pinskia at gcc dot gnu dot org 2006-10-22 23:01 ---
Fixed in 4.0 by the tree-ssa merge.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
long foo( bool flag, int __x )
{
return ( flag ? (unsigned)__x : __x ); // broken
}
long bar( bool flag, int __x )
{
if ( flag )
return (unsigned)__x;
else
return __x;
}
:
0: 89 f0 mov%esi,%eax
--- Comment #1 from andreast at gcc dot gnu dot org 2006-10-22 20:27
---
Fixed with this commit:
http://gcc.gnu.org/ml/java-patches/2006-q4/msg00084.html
Confirmed here:
http://gcc.gnu.org/ml/gcc-testresults/2006-10/msg01120.html
This also fixes events run test on linux-ppc 32-bit.
--- Comment #6 from burnus at net-b dot de 2006-10-22 21:03 ---
See http://gcc.gnu.org/ml/gcc-patches/2006-10/msg01086.html for a preliminary
patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24790
--- Comment #3 from pinskia at gcc dot gnu dot org 2006-10-22 21:01 ---
On the mainline we get:
subl$12, %esp
fld1
faddl 16(%esp)
fstpl (%esp)
movl4(%esp), %eax
addl$12, %esp
andl$-2147483648, %eax
ret
W
--- Comment #1 from pinskia at gcc dot gnu dot org 2006-10-22 21:02 ---
(In reply to comment #0)
> (BTW, none of the product categories seem right, but I was directed here by
> the
> manual of gnu as.)
The correct place is http://sourceware.org/bugzilla/ under binutils.
This bugzilla
The following bit of (malformed) assembly causes GNU as to crash.
.globl foo
.type foo, @function
foo:
sete$%al
Here is the error message:
$ as test.s
test.s: Assembler messages:
test.s:4: Internal error, aborting at ../../gas/config/tc-i386.c line 2123 in
optimize_imm
Pleas
--- Comment #32 from rguenth at gcc dot gnu dot org 2006-10-22 20:16
---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNE
--- Comment #5 from kkojima at gcc dot gnu dot org 2006-10-23 00:28 ---
Fixed.
--
kkojima at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2006-10-22 18:45
---
The current code already recognizes matrix transposition and gives BLAS gemm
functions the right TRANSA and TRANSB argument in this case.
Confirmed for CONJG, which we don't currently handle.
--
fxcoudert at
--- Comment #12 from ebotcazou at gcc dot gnu dot org 2006-10-22 21:27
---
> My point is that the subprogram I wrote is probably the way most Ada
> programmers will write when they want to multiply matrices (if not using
> BLAS/Lapack).
OK, but you have not simplified the problem in do
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Ke
--- Comment #2 from rguenth at gcc dot gnu dot org 2006-10-22 20:20 ---
In both cases we now get a call to g(1) due to the fix for 15911. So, fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #31 from rguenth at gcc dot gnu dot org 2006-10-22 20:11
---
Subject: Bug 15911
Author: rguenth
Date: Sun Oct 22 20:11:09 2006
New Revision: 117960
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117960
Log:
2006-10-22 Jeff Law <[EMAIL PROTECTED]>
Richard G
--- Comment #2 from brandon at rioja dot us 2006-10-22 18:09 ---
Ahh, I see. This document says that the __gnu_cxx namespace is for the C++
extentions for public use:
http://gcc.gnu.org/onlinedocs/libstdc++/latest-doxygen/doxygroups_8cc-source.html
"""
This namespace serves two purposes
--- Comment #10 from ebotcazou at gcc dot gnu dot org 2006-10-22 18:33
---
> I don't know how to write the equivalent fortran code, but it's likely better
> to concentrate on the subprogram version for optimization purpose.
Sorry, I don't see your point.
--
http://gcc.gnu.org/bugz
--- Comment #5 from dberlin at gcc dot gnu dot org 2006-10-22 17:39 ---
(In reply to comment #1)
> I think it's ok again, but it shows one weakness of the current NONLOCAL
> scheme:
>
> foo ()
> {
> struct Foo a;
> int D.1534;
>
> :
> # SFT.0_2 = V_MUST_DEF ;
> a.i = 1;
> #
--- Comment #4 from dberlin at gcc dot gnu dot org 2006-10-22 17:37 ---
Subject: Re: FAIL: gcc.dg/tree-ssa/pr26421.c scan-tree-dump-times V_MAY_DEF 1
On 22 Oct 2006 17:16:00 -, rguenth at gcc dot gnu dot org
<[EMAIL PROTECTED]> wrote:
>
>
> --- Comment #2 from rguenth at gcc do
--- Comment #3 from pluto at agmk dot net 2006-10-22 17:44 ---
recent 4.2 produces worse %rcx setup code.
previously:
xorl%edx, %edx
movq%rdx, %rcx
now:
movq%rdi, %rax
sarq$63, %rax
movq%rax, %rcx
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26674
--- Comment #3 from pinskia at gcc dot gnu dot org 2006-10-22 17:30 ---
http://gcc.gnu.org/ml/gcc-cvs/2006-10/msg00625.html
That just needs to merged back.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29551
Sees:
In power() is not declared in the std namespace, and the program
doesn't compile.
Expects:
power to be included in the namespace.
Source:
using namespace std;
#include
#include
int main() {
cout << power (2,30) << endl;
}
Command line Flag and Compiler error:
[EMAIL PROTECTED] testp
--- Comment #1 from rguenth at gcc dot gnu dot org 2006-10-22 17:12 ---
I think it's ok again, but it shows one weakness of the current NONLOCAL
scheme:
foo ()
{
struct Foo a;
int D.1534;
:
# SFT.0_2 = V_MUST_DEF ;
a.i = 1;
# SFT.0_6 = V_MAY_DEF ;
# NONLOCAL.6_7 = V_M
--- Comment #2 from rguenth at gcc dot gnu dot org 2006-10-22 17:16 ---
Adding
p = &a
and using that produces
foo ()
{
struct Foo * p;
struct Foo a;
int D.1535;
:
# SFT.2_2 = V_MUST_DEF ;
a.i = 1;
p_3 = &a;
# SFT.0_9 = V_MAY_DEF ;
# SFT.1_10 = V_MAY_DEF ;
#
--- Comment #1 from pinskia at gcc dot gnu dot org 2006-10-22 17:35 ---
Right because this power is an extension to the C++ standard.
This is not a bug.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
This test has regressed.
Executing on host: /home/dave/gnu/gcc-4.3/objdir/gcc/xgcc
-B/home/dave/gnu/gcc-4
.3/objdir/gcc/
/home/dave/gnu/gcc-4.3/gcc/gcc/testsuite/gcc.dg/tree-ssa/pr26421.
c -O2 -fdump-tree-alias1-vops -fno-show-column -S -o pr26421.s(timeout =
300)
PASS: gcc.dg/tree-ssa/pr26
--- Comment #10 from pluto at agmk dot net 2006-10-22 17:17 ---
i will no longer test the 4.1 branch.
for me it's closed with /dontcare.
--
pluto at agmk dot net changed:
What|Removed |Added
-
--- Comment #4 from kkojima at gcc dot gnu dot org 2006-10-23 00:14 ---
Subject: Bug 28232
Author: kkojima
Date: Mon Oct 23 00:14:39 2006
New Revision: 117961
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117961
Log:
PR target/28232
* config/sh/sh.c (expand_bloc
Often, matrix multiplications contain transpose() or conj(), e.g.
matmul(transpose(A),B)
or
matmul(A,conj(transpose(B))
matmul(A,transpose(conj(B))
The *gemm subroutines of BLAS anticipate this via the TRANSA and TRANSB
options:
- 'N' (unchanged)
- 'T' (transpose)
- 'C' (hermitian conjugate
--- Comment #9 from dave at hiauly1 dot hia dot nrc dot ca 2006-10-22
16:18 ---
Subject: Re: FAIL: gfortran.dg/actual_array_constructor_1.f90 -O0 (ICE)
> --- Comment #8 from sje at cup dot hp dot com 2006-10-19 18:09 ---
> Well, I found that the TImode is getting introduced
--- Comment #9 from laurent at guerby dot net 2006-10-22 16:13 ---
Eric, build is "i686-pc-linux-gnu" I don't understand why you speak of
generalization :).
Anyway, I changed the code to match more likely use, that is by using a
subprogram:
procedure Compute (A, B : in Real_Matrix;
--- Comment #15 from nathan at gcc dot gnu dot org 2006-10-22 16:09 ---
2006-10-22 Nathan Sidwell <[EMAIL PROTECTED]>
PR c++/20647
* rtti.c (tinfo_base_init): The type info string is always global.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20647
--- Comment #14 from nathan at gcc dot gnu dot org 2006-10-22 16:07 ---
Subject: Bug 20647
Author: nathan
Date: Sun Oct 22 16:07:41 2006
New Revision: 117957
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117957
Log:
cp/
PR c++/20647
* rtti.c (tinfo_base_init): T
Comparison by Francois-Xavier:
http://www.eleves.ens.fr/home/coudert/timing.png
(bottom row: complex numbers; black gfortran matmul, others: optimized
libraries).
See also http://gcc.gnu.org/ml/gcc-patches/2006-10/msg00343.html
--
Summary: matmul slow for complex matrices
--- Comment #1 from rguenth at gcc dot gnu dot org 2006-10-22 15:56 ---
I have a patch.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|
fold_plusminus_mult_expr does not fold
int f6 (int a, int b)
{
return b * -2 + a * 6;
}
this blocks the fix for PR23295 because a fix for it makes the test for
PR23294 fail.
--
Summary: Does not fold b * -2 + a * 6
Product: gcc
Version: 4.3.0
Stat
--- Comment #8 from jeff at thecreems dot com 2006-10-22 15:50 ---
(In reply to comment #7)
> > No, but there is an obvious one why the Fortran version trounces the Ada
> > version. Let's not compare apples with oranges.
>
> The adverse effect of the indirection can be alleviated thoug
--- Comment #10 from rguenth at gcc dot gnu dot org 2006-10-22 15:12
---
With the patch for 23295 we get the behavior as noted in comment #7. This is
due
to negate_expr_p (correctly) containing
case BIT_NOT_EXPR:
return INTEGRAL_TYPE_P (type)
&& (TYPE_UNSIGNED
--- Comment #7 from ebotcazou at gcc dot gnu dot org 2006-10-22 15:03
---
> No, but there is an obvious one why the Fortran version trounces the Ada
> version. Let's not compare apples with oranges.
The adverse effect of the indirection can be alleviated though:
N : Positive := Po
--- Comment #7 from sgk at troutmask dot apl dot washington dot edu
2006-10-22 15:00 ---
Subject: Re: ICE in gfc_match_common for blank common in BLOCK DATA unit
On Sun, Oct 22, 2006 at 11:36:15AM -, aldot at gcc dot gnu dot org wrote:
>
> --- Comment #5 from aldot at gcc dot
--- Comment #9 from rguenth at gcc dot gnu dot org 2006-10-22 14:45 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-10-22 14:15 ---
Note if you add -pedantic or -ansi or -std=c99 or -std=c89, powerpc will not be
defined as it is not in the reserved space.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29546
--- Comment #1 from rguenth at gcc dot gnu dot org 2006-10-22 13:21 ---
powerpc is one of the standard defines gcc defines if running on ppc-linux. It
is changed to __powerpc if you define powerpc yourself.
--
rguenth at gcc dot gnu dot org changed:
What|Removed
--- Comment #2 from joseph at codesourcery dot com 2006-10-22 13:20 ---
Subject: Re: Enable IBM long double format in PowerPC
Linux soft float
On Sat, 21 Oct 2006, pinskia at gcc dot gnu dot org wrote:
> There was talking about this on the mailing list but I cannot find it right
> no
When you want to set the "powerpc" string as a value of a define (-D or
#define), you get the integer value 1 instead of the string.
Here's an example :
test.c :
-
Arch = ARCH
-
With the "powerpc" string (bug) :
gcc -E -DARCH=powerpc test.c
Result is :
# 1 "test.c"
# 1 ""
# 1 ""
# 1 "te
--- Comment #3 from falk at debian dot org 2006-10-22 12:50 ---
This bug is still there in 4.1, but not in mainline.
Janis, can you perhaps do a regression hunt to see what fixed this and whether
it is feasible to backport?
--
falk at debian dot org changed:
What|Rem
--- Comment #6 from ebotcazou at gcc dot gnu dot org 2006-10-22 12:42
---
> Eric, if the type is Long_Float your solution will be much worse since the
> alignement on the stack of the matrices will be 4-bytes so you have 50% chance
> the code will be N times slower (when not 8-bytes ali
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2006-10-22 12:26
---
Fixed in 4.2 and later, thanks to Tobias.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #5 from laurent at guerby dot net 2006-10-22 12:04 ---
Eric, if the type is Long_Float your solution will be much worse since the
alignement on the stack of the matrices will be 4-bytes so you have 50% chance
the code will be N times slower (when not 8-bytes aligned).
So in
--- Comment #6 from aldot at gcc dot gnu dot org 2006-10-22 11:50 ---
> I think the code in comment #1 is valid.
Rejecting blank common in block data stmts would add a regression in
testsuite's blockdata_1.f90 like so:
[EMAIL PROTECTED]:~/src/gcc-4.2/gcc/testsuite/gfortran.dg$ gfortran-
--- Comment #5 from aldot at gcc dot gnu dot org 2006-10-22 11:36 ---
I think the code in comment #1 is valid.
f95, 5.5.2.4 Differences between named common and blank common
reads:
A blank common block has the same properties as a named common block except
(iii) [...] objects in blank c
--- Comment #7 from richard at codesourcery dot com 2006-10-22 08:51
---
Subject: Re: [4.2 Regression] Bad code on MIPS with -fnon-call-exceptions
"daney at gcc dot gnu dot org" <[EMAIL PROTECTED]> writes:
> Richard, does this look right?
Looks good to me.
--
http://gcc.gnu.org/
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:45
---
I don't think we want to have a real .mod file somewhere, but I know how I'll
implement it internally. I'll do it.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:43
---
Fixed on mainline.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:42
---
Subject: Bug 26025
Author: fxcoudert
Date: Sun Oct 22 07:41:48 2006
New Revision: 117948
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117948
Log:
PR fortran/26025
* lang.opt: Add -fexte
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:19
---
Confirmed.
This one will probably be implemented after ISO_C_BINDING.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:18
---
No, it's implemented, it's only that the subroutine and function with the same
name cannot be called in the same scoping unit.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2006-10-22 07:15
---
(In reply to comment #1)
> - A compile-time-error is shown (probably fixed by FX patch, not checked)
Yes, it's fixed by my patch. Confirming this bug.
Tobias, if you want to submit a global patch to resolve thes
81 matches
Mail list logo
