--- Comment #8 from burnus at gcc dot gnu dot org 2007-10-22 06:56 ---
(In reply to comment #6)
> Created an attachment (id=14381)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14381&action=view) [edit]
> Patch to add checks for double specifics
This looks wrong:
+gfc_check_a_pd (
--- Comment #5 from ubizjak at gmail dot com 2007-10-22 06:45 ---
I'm testing following patch:
--cut here--
Index: combine.c
===
--- combine.c (revision 129547)
+++ combine.c (working copy)
@@ -9386,7 +9386,9 @@ simplif
--- Comment #3 from tbm at cyrius dot com 2007-10-22 06:37 ---
Breakpoint 1, fancy_abort (file=0xc4e238 "gcc/tree-vect-transform.c",
line=3374, function=0xc4fa60 "vectorizable_conversion")
at gcc/diagnostic.c:659
659 {
(gdb) where
#0 fancy_abort (file=0xc4e238 "gcc/tree-vect
--- Comment #2 from tbm at cyrius dot com 2007-10-22 06:36 ---
/* Testcase by Martin Michlmayr <[EMAIL PROTECTED]> */
extern void *malloc (long unsigned int __size);
typedef struct VMatrix_ VMatrix;
struct VMatrix_
{
int dim;
int t2;
};
void uniform_correlation_matrix (VMatrix * v)
--- Comment #1 from tbm at cyrius dot com 2007-10-22 06:35 ---
Created an attachment (id=14385)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14385&action=view)
preprocessed source
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33854
I see the following ICE with current trunk. I can also reproduce this
problem with trunk from 2007-06-04, but not with 2007-05-15.
(sid)29918:[EMAIL PROTECTED]: ~] /usr/lib/gcc-snapshot/bin/gcc -O1
-ftree-vectorize -c
gretl-describe.c
gretl-describe.c: In function 'uniform_correlation_matrix':
g
--- Comment #7 from irar at il dot ibm dot com 2007-10-22 06:31 ---
Falk,
Could you please check if the patch in Comment #6 fixes the ICE?
Thanks,
Ira
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33804
--- Comment #2 from burnus at gcc dot gnu dot org 2007-10-22 06:22 ---
> Is the issue here only the wording of the error message
Yes, only the message text.
"Error: GENERIC non-INTRINSIC procedure '%s' is not allowed as an actual
argument"
should be
"Error: GENERIC procedure '%s' is
--- Comment #4 from dorit at gcc dot gnu dot org 2007-10-22 04:37 ---
I'm testing a patch that would fix both this PR and PR33834 (posted it under
the PR33834 entry). By the way, this testcase does not get vectorized with
current mainline (an Oct21 snapshot) because the call to cos is no
--- Comment #6 from dorit at gcc dot gnu dot org 2007-10-22 04:28 ---
I'm testing this patch. It fixes the two testcases, while allowing the first
testcase to get vectorized. (the last bit in the patch is the fix for PR33835):
Index: tree-vect-analyze.c
=
--
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
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
CC||pinskia at gcc dot gnu dot
|
--- Comment #5 from dave at hiauly1 dot hia dot nrc dot ca 2007-10-22
01:30 ---
Subject: Re: gcc.c-torture/execute/longlong.c
execution at -O3
Things appear to go wrong in the greg pass:
(insn 73 133 70 4
/home/dave/gnu/gcc-4.3/gcc/gcc/testsuite/gcc.c-torture/execute
/longlon
--- Comment #4 from danglin at gcc dot gnu dot org 2007-10-22 00:43 ---
The actual code that is wrong is the following:
0x00010670 : ldw 10(r20),r19
0x00010674 : uaddcm r0,r23,r21
0x00010678 : uaddcm r0,r24,r22
0x0001067c : depw,z r19,30,31,r19
0x00010680 : subi 1f,r22,r21
0x000106
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2007-10-21 22:56
---
Is the issue here only the wording of the error message or is the checking
incorrect as well?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33849
Here is incorrect text formating with using cout() with setw() in UNICODE
(UTF-8)
Two text: first in English - correct formating, second - russian in UTF-8 -
incorrect.
#include
#include
int main() {
std::cout.setf(std::ios::left);
std::cout.fill('_');
std::cout << std:
--- Comment #1 from hjl at lucon dot org 2007-10-21 21:45 ---
I believe we should add those deleted ext/xxx header files with
#warning __FILE__ is deprecated
#include
if we want to move ext/xxx to backward/xxx. It is never a good idea to break
the existing C++ applications for no
--- Comment #3 from hjl at lucon dot org 2007-10-21 21:41 ---
I believe all those header files should be restored unless we have a very
good reason to break libstdc++ API.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33831
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2007-10-21 21:32
---
Created an attachment (id=14382)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14382&action=view)
Preliminary argument checking with intrinsics
This "draft" patch catches the latest invalid case provided in
--- Comment #16 from tobi at gcc dot gnu dot org 2007-10-21 21:18 ---
Re this testcase:
(In reply to comment #5)
> program array_char
> implicit none
> character (len=2) :: x, y
> character (len=2) :: z(2)
> x = "a "
> y = "cd"
> z = (/y(1:len(trim(y))), x(1:len(trim(x)))/) ! causes seg
--- Comment #15 from Tobias dot Schlueter at physik dot uni-muenchen dot de
2007-10-21 20:13 ---
Subject: Re: Diagnose different string lengths in array
constructors at run time
dominiq at lps dot ens dot fr wrote:
> --- Comment #14 from dominiq at lps dot ens dot fr 2007-10-21
--- Comment #14 from dominiq at lps dot ens dot fr 2007-10-21 20:05 ---
Now I understand the strange result I reported in comment #5. What is happening
is that there is a second nonprintable character. If I redirect the output to a
file I see it (in general ^@).
>From what I understand
--- Comment #3 from benoit dot hudson at gmail dot com 2007-10-21 18:23
---
(In reply to comment #2)
> Well, libtool needs to check the gcc version.
I fully understand that morally, it's libtool's fault for releasing a program
that didn't correctly divine that gcc 4.3 would change the
--- Comment #6 from pault at gcc dot gnu dot org 2007-10-21 18:10 ---
Subject: Bug 33749
Author: pault
Date: Sun Oct 21 18:10:00 2007
New Revision: 129539
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129539
Log:
2007-10-21 Paul Thomas <[EMAIL PROTECTED]>
PR fortran/
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
CC||pinskia at gcc dot gnu dot
|
--- Comment #2 from pinskia at gcc dot gnu dot org 2007-10-21 16:45 ---
Well, libtool needs to check the gcc version. The issue is that in 4.3, we
call ld directly instead of libtool (Apple's libtool and not GNU libtool). So
libtool needs to use dylib_compatibility_version/dylib_curren
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2007-10-21 16:39
---
Created an attachment (id=14381)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14381&action=view)
Patch to add checks for double specifics
This patch adds checks for double specific intrinsics. I added che
--- Comment #2 from oliver dot kellogg at eads dot com 2007-10-21 15:44
---
Also happens with 4.3.0-20071021 r129524.
Does not happen with -O (optimize).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33788
--- Comment #1 from benoit dot hudson at gmail dot com 2007-10-21 15:15
---
I should specify: I can *compile* but I can't *link*.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33851
libtool adds all sorts of wonderful flags to gcc, whose purpose is unclear.
Nonetheless, the following works on gcc-4.2.1 but fails on gcc version 4.3.0
20071012 (experimental) (GCC) on the same box, a Mac Pro running OS X 10.4:
g++ -m64 -dynamiclib -single_module ${wl}-flat_namespace ${wl}-undef
--- Comment #6 from irar at il dot ibm dot com 2007-10-21 12:52 ---
The solution can be just not check if the vectorization is worthwhile during
the transformation. The decision whether to vectorize or not should be made
during the analysis anyway.
The vectorization factor can get small
--- Comment #1 from burnus at gcc dot gnu dot org 2007-10-21 12:40 ---
See also http://gcc.gnu.org/ml/fortran/2007-10/msg00298.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33850
For
integer(4) :: p(4) = (/2,4,1,3/)
integer(4) :: q(4) = (/2,4,1,3/)
p(q) = (/(i, i = 1, 4)/)
gfortran generates an unneeded temporary, which is only needed for
integer(4) :: p(4) = (/2,4,1,3/)
p(p) = (/(i, i = 1, 4)/)
Cf. PR 33749
--
Summary: Unneeded temporary gene
--- Comment #14 from pcarlini at suse dot de 2007-10-21 11:33 ---
(In reply to comment #13)
> Sure thing.
Excellent.
> I will need a little bit of time to familiarize myself with the current code.
> Also, I will need a little bit of help in finding my way around. E.g., where
> do
> th
--- Comment #4 from irar at il dot ibm dot com 2007-10-21 11:02 ---
The problem is with vector shift with scalar shift argument.
For the code created by the vectorizer:
vect_var_.49_103 = ~vect_var_.47_101;
vect_var_.50_105 = vect_var_.49_103 >> 31;
(ashiftrt:V4SI (not:V4SI (reg:V4
--- Comment #5 from burnus at gcc dot gnu dot org 2007-10-21 10:44 ---
> This patch: Allows the test case in comment #1 to compile and run correctly
> with
> no regressions in the test suite.
It also wrongly allows the test case in the bugreport PR 33847 to compile. Thus
we should add
--- Comment #5 from tbm at cyrius dot com 2007-10-21 10:44 ---
Trying to reduce...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33848
The following error message is misleading:
"Error: GENERIC non-INTRINSIC procedure '%s' is not allowed as an actual
argument"
The problem is that any GENERIC procedure - intrinsic or not - is not allowed
as actual argument. (For "a dummy procedure[...], the associated actual
argument shall be the
--- Comment #2 from burnus at gcc dot gnu dot org 2007-10-21 10:33 ---
I cannot read. You used:
interface shape_line_P1
module procedure shape_line_P1, shape_line_P1_2
end interface
Thus "shape_line_P1" is both a specific subroutine and a generic subroutine,
however, the standa
--- Comment #4 from tbm at cyrius dot com 2007-10-21 10:15 ---
(In reply to comment #3)
> This will help you find the pass during/after which the label disappears.
It's there from 104r (expand) to 139r (postreload) but not anymore in 141r
(flow2).
--
http://gcc.gnu.org/bugzilla/sho
--- Comment #1 from burnus at gcc dot gnu dot org 2007-10-21 10:07 ---
(And the program is accepted using the patch from bug 33162 comment 2, however,
I think the patch is the wrong to solution for this PR and bug 33162.)
NAG f95 also gives the following error:
Error: line 41: Actual p
--- Comment #3 from steven at gcc dot gnu dot org 2007-10-21 10:00 ---
The way for you to narrow down the bug is this:
1. Compile with -daAP
2. Look in the .s file which instruction references the missing label. There
should be a LABEL_REF with a number.
3. Grep for "code_label.*" in th
--- Comment #13 from jkherciueh at gmx dot net 2007-10-21 09:49 ---
(In reply to comment #12)
> In order to have a fully conforming implementation, we need something which
> works for *any* source range and *any* target range. Are you willing to
> work on that? Would be mainline-only of
--- Comment #2 from tbm at cyrius dot com 2007-10-21 09:46 ---
Created an attachment (id=14380)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14380&action=view)
preprocessed source
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33848
--- Comment #1 from tbm at cyrius dot com 2007-10-21 09:46 ---
The problem is that gcc-4.2 generates a bne instruction that references a
label that doesn't exist. This happens with -O1 or higher. It doesn't
happen with gcc 4.1 or current trunk.
If you compile the attached preprocessed
[ Forwarded from http://bugs.debian.org/441633 ]
From: Sergei Golovan <[EMAIL PROTECTED]>
Current gcc 4.2 fails to build erlang package on mips and mipsel
architectures. It works on all other architectures.
Linker error messages are the following (the same for both mips and mipsel):
obj/mipsel-u
--- Comment #4 from burnus at gcc dot gnu dot org 2007-10-21 09:30 ---
Several intrinsic functions come as specific and as generic functions. For
instance
COS as generic function takes as argument REAL(k) and COMPLEX(k), k=4,8,10,16
and returns the same type and kind as it got as
--- Comment #5 from irar at il dot ibm dot com 2007-10-21 08:45 ---
(In reply to comment #4)
> Created an attachment (id=14370)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14370&action=view) [edit]
> Vectorization dump file
>
Thanks!
The vectorizer fails in transformation phase
--- Comment #3 from dorit at gcc dot gnu dot org 2007-10-21 08:07 ---
The proposed fix/work-around for PR33834 also happens to fix this PR. But the
real problem is that we try to access a NULL argument (operand 2 of a CALL_EXPR
may be NULL). So we should probably at least add something l
--- Comment #2 from pault at gcc dot gnu dot org 2007-10-21 07:30 ---
Given Steve's comment, this clearly should be confirmed!
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #5 from dorit at gcc dot gnu dot org 2007-10-21 07:14 ---
This patch fixes it:
Index: tree-vect-transform.c
===
*** tree-vect-transform.c (revision 129521)
--- tree-vect-transform.c (working copy)
**
51 matches
Mail list logo
