from linux-2.6.36-rc3, `fs/jdb2/journal.c' triggers the following:
/linux-2.6/fs/jbd2/journal.c:1200:1: internal compiler error: Segmentation
fault
Please submit a full bug report,
with preprocessed source if appropriate.
See http://gcc.gnu.org/bugs.html for instructions.
On a reduced testcase:
--- Comment #1 from lacombar at gmail dot com 2010-09-08 06:13 ---
Created an attachment (id=21734)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21734action=view)
reduced testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45593
--- Comment #19 from burnus at gcc dot gnu dot org 2010-09-08 06:25 ---
Reviewed patch (OK) available at
http://gcc.gnu.org/ml/fortran/2010-09/msg00198.html
however, it causes regressions as some of the intrinsics (in intrinsic.c) have
the wrong intents - which causes wrong code (too
--- Comment #4 from dominiq at lps dot ens dot fr 2010-09-08 06:55 ---
This pr could be a duplicate of pr45578.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45589
--- Comment #6 from dominiq at lps dot ens dot fr 2010-09-08 06:56 ---
pr45589 could be a duplicate of this pr.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45578
--- Comment #3 from gingold at gcc dot gnu dot org 2010-09-08 07:27 ---
Subject: Bug 44001
Author: gingold
Date: Wed Sep 8 07:27:11 2010
New Revision: 163989
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163989
Log:
2010-09-08 Tristan Gingold ging...@adacore.com
PR
--- Comment #4 from gingold at gcc dot gnu dot org 2010-09-08 08:26 ---
Subject: Bug 44001
Author: gingold
Date: Wed Sep 8 08:25:39 2010
New Revision: 163993
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163993
Log:
2010-09-08 Tristan Gingold ging...@adacore.com
PR
--- Comment #5 from gingold at gcc dot gnu dot org 2010-09-08 08:48 ---
Fixed with the filed patches.
--
gingold at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from ibolton at gcc dot gnu dot org 2010-09-08 08:49 ---
(In reply to comment #6)
(In reply to comment #5)
Do we need to act as if
-fno-ira-share-spill-slots
is set in cfun-calls_setjmp functions?
At least in my case -Os -fno-ira-share-spill-slots seems to solve
If two separate .cpp files contain two versions of inline function with the
same signature, that are used by functions in the corresponding files, one of
them is dropped and all functions use only one of them.
Consider example:
main.cpp
-
#include test1.h
#include
--- Comment #3 from ibolton at gcc dot gnu dot org 2010-09-08 09:12 ---
Thanks for raising this bug, Gergely, and suggesting a patch. I've moved the
bug to the NEW state, so if you want to post your patch to the gcc-patches
list, then you will hopefully get some feedback on it there
--- Comment #5 from ubizjak at gmail dot com 2010-09-08 09:18 ---
(In reply to comment #1)
and search for the line with starts with /some/path/f95 and run then the
f951
line in a debugger, e.g.
gdb --args /some/path/to/f951 many, many, many options
and then in (gdb)
run
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-09-08 09:20 ---
You are violating the ODR.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from iains at gcc dot gnu dot org 2010-09-08 09:20 ---
also on powerpc64-darwin9
--
iains at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-09-08 09:22 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-09-08 09:22 ---
Subject: Bug 45590
Author: rguenth
Date: Wed Sep 8 09:22:35 2010
New Revision: 163995
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163995
Log:
2010-09-08 Richard Guenther rguent...@suse.de
PR
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-09-08 09:22 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from mikpe at it dot uu dot se 2010-09-08 09:30 ---
I can reproduce the ICE on sparc64-linux with 4.5-20100902 and 4.6-20100904 -Os
-m32. With -m64 or -O1/O2/O3 it doesn't ICE. 4.4-20100817 doesn't ICE.
--
mikpe at it dot uu dot se changed:
What
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-09-08 09:36 ---
Confirmed. We have two different array3_real(kind=8) record types that are
not considered compatible. One data pointer member is restrict qualified
while the other one is not.
Why do we have an aggregate
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.6.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45585
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-09-08 09:37 ---
Confirmed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from janus at gcc dot gnu dot org 2010-09-08 09:49 ---
This problem is apparently related to type extension (but not to abstract
types). The following still fails:
implicit none
type :: pointGen
end type pointGen
type, extends(pointGen) :: point2d
--- Comment #21 from abnikant dot singh at atmel dot com 2010-09-08 09:50
---
The head version [gcc version 4.6.0 20100907 (experimental) (GCC)] tends to
inline the attached test case in case of -Os, just because it gets better code
size [see the dump using : -fdump-ipa-inline] by
--- Comment #10 from paolo dot carlini at oracle dot com 2010-09-08 09:56
---
(In reply to comment #8)
Maybe you should tell that to Paolo Carlini, who closed bug 15002 as resolved
fixed in 2004,
And it *is* fixed. Did you actually open the testcases? Just plain fstreams,
thus no
--- Comment #11 from paolo dot carlini at oracle dot com 2010-09-08 09:59
---
(In reply to comment #8)
But a 9-10x difference doesn't sound reasonable to me. The synced mode is not
unbuffered, before or after my suggested change, it uses the internal buffer
in
glibc.
So? We are
--- Comment #5 from ibolton at gcc dot gnu dot org 2010-09-08 10:02 ---
Confirmed on latest 4.4, 4.5 and 4.6 (trunk).
Related GCC documentation on alignment of structure fields is here:
http://gcc.gnu.org/onlinedocs/gcc-4.5.0/gcc/Variable-Attributes.html#Variable-Attributes
In the
--- Comment #23 from ibolton at gcc dot gnu dot org 2010-09-08 10:06
---
(In reply to comment #21)
Subject: Re: Make fails in zlib
Hello;
Well I solved my problem, however the issue still remains. I installed
the latest native binutils and gcc-4.5.1 on my linux
--- Comment #6 from mikpe at it dot uu dot se 2010-09-08 10:16 ---
(In reply to comment #5)
I can't get this to ICE with latest version of 4.4, 4.5 or 4.6 branches.
The default_secondary_reload ICE is on a gcc_assert() so you must configure
with --enable-checking;
--- Comment #12 from paolo dot carlini at oracle dot com 2010-09-08 10:20
---
By the way, getdelim is not standard, thus would work only on linux, even more
special casing. More importantly, fgets *stores* newline and '\0', at variance
with getline, I don't think it can be used as-is
--- Comment #2 from justchecking8964 at gmail dot com 2010-09-08 10:30
---
(In reply to comment #1)
You are violating the ODR.
ODR rule relates only to the non-inline functions, which is not the case here,
see http://en.wikipedia.org/wiki/One_Definition_Rule :
#2: In the entire
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-09-08 10:34 ---
(In reply to comment #2)
The case that apply here is discussed in point 3:
#3: ... For a given entity, each definition
must be the same. ...
--
rguenth at gcc dot gnu dot org changed:
What
--- Comment #4 from justchecking8964 at gmail dot com 2010-09-08 10:39
---
(In reply to comment #3)
(In reply to comment #2)
The case that apply here is discussed in point 3:
#3: ... For a given entity, each definition
must be the same. ...
But this applies only to [...]
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-09-08 10:42 ---
(In reply to comment #4)
(In reply to comment #3)
(In reply to comment #2)
The case that apply here is discussed in point 3:
#3: ... For a given entity, each definition
must be the same. ...
--- Comment #6 from justchecking8964 at gmail dot com 2010-09-08 11:00
---
Your inline functions are extern, not static. Make them static and it
will work.
I see, thanks, I missed the part of the new standard that defines inline as
being implicitly extern. There's something new
--- Comment #8 from christian dot eggers at kathrein dot de 2010-09-08
11:12 ---
(In reply to comment #7)
Is this still a bug then? Should ira-share-spill-slots be automatically
disabled for the caller function when a callee function can return twice?
I've never tested with
--- Comment #7 from rguenth at gcc dot gnu dot org 2010-09-08 11:17 ---
Subject: Bug 45578
Author: rguenth
Date: Wed Sep 8 11:17:31 2010
New Revision: 163997
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163997
Log:
2010-09-08 Richard Guenther rguent...@suse.de
PR
--- Comment #8 from rguenth at gcc dot gnu dot org 2010-09-08 11:21 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #27 from dominiq at lps dot ens dot fr 2010-09-08 11:29 ---
What is the fate of the patch in comment #19?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43829
this test case (with an invalid collapse):
SUBROUTINE init_input_type()
INTEGER :: k,l(3),u(3)
!$omp parallel do shared(l,u) collapse(3)
DO k = l(3),u(3)
ENDDO
END SUBROUTINE
leads to an ice with current trunk as:
#0 gfc_resolve_omp_directive (code=0x1437880, ns=value optimized
This is a RFC, the intent of this is to improve tonto speed and hopefully other
Fortran testcases.
I'll attach a WIP patch.
What it currently does is from frontend-passes.c walks down procedure body and
tries to determine all possible POINTER assignments (currently for POINTER vars
only, no
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-08 12:01 ---
Created an attachment (id=21735)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21735action=view)
gcc46-pr45596.patch
WIP patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45596
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-08 12:05 ---
Created an attachment (id=21736)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21736action=view)
uu.f90
One testcase I've been playing with. The patch allows optimizing away
temporaries on 3 of the 4 p? = a + ?
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-08 12:08 ---
Created an attachment (id=21737)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21737action=view)
tonto2.f90
Modified testcase from tonto (well, in particular from richi's testcase in one
of the tonto PRs), where
gfortran -c -fopenmp bug.f90
bug.f90: In function rs_pw_transfer_distributed:
bug.f90:6:0: internal compiler error: in gfc_trans_cycle, at
fortran/trans-stmt.c:4320
Please submit a full bug report,
with preprocessed source if appropriate.
See http://gcc.gnu.org/bugs.html for instructions.
--
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
--
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 #7 from paolo dot carlini at oracle dot com 2010-09-08 12:21
---
new?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45594
--- Comment #6 from mikpe at it dot uu dot se 2010-09-08 12:24 ---
The smallest .o file that differs between stage2 and stage3 is sreal.o. Diffing
the objdump -d output shows:
@@ -1,5 +1,5 @@
-stage2-gcc/sreal.o: file format elf32-littlearm
+stage3-gcc/sreal.o: file format
--- Comment #3 from matz at gcc dot gnu dot org 2010-09-08 12:35 ---
Subject: Bug 33244
Author: matz
Date: Wed Sep 8 12:34:52 2010
New Revision: 163998
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163998
Log:
PR tree-optimization/33244
* tree-ssa-sink.c
--- Comment #8 from matz at gcc dot gnu dot org 2010-09-08 12:40 ---
Subject: Bug 43430
Author: matz
Date: Wed Sep 8 12:40:24 2010
New Revision: 163999
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163999
Log:
PR tree-optimization/43430
* tree-vect-stmts.c
--- Comment #9 from ebotcazou at gcc dot gnu dot org 2010-09-08 12:54
---
*** Bug 45421 has been marked as a duplicate of this bug. ***
--
ebotcazou at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from ebotcazou at gcc dot gnu dot org 2010-09-08 12:54
---
Unfortunately, even with your patch the mips-sgi-irix6.5 Ada bootstrap
is still broken.
OK, thanks for testing. Presumably fixed by Richard now, reopen if not.
*** This bug has been marked as a duplicate of
Between revisions 163966 (working) and 164000 compiling the following test
gives an ICE with -O[23s]
[macbook] f90/bug% cat pr30940.f90
program main
implicit none
character(len=10) :: digit_string = '123456789'
character :: digit_arr(10)
call copy(digit_string, digit_arr)
print '(1x,
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-09-08 13:53 ---
#4 0x00b25f5e in fold_const_aggregate_ref (t=0x77f0cc18)
at /space/rguenther/src/svn/trunk/gcc/tree-ssa-ccp.c:1444
1444return build_int_cst_type (TREE_TYPE (t),
(gdb) l
1439
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-08 14:02 ---
Created an attachment (id=21738)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21738action=view)
gcc46-pr45595.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45595
The pragma once is obsolete due to cpp optimizations and #pragma once can
be completely ignored. Any code referring to it should be removed.
--
Summary: Remove all code applying to obsolete #pragma once
Product: gcc
Version: unknown
Status:
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-08 14:25 ---
Created an attachment (id=21739)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21739action=view)
gcc46-pr45597.patch
Seems to be a recent regression, caused by
--- Comment #1 from pinskia at gcc dot gnu dot org 2010-09-08 14:46 ---
#pragma once
Can you explain why you think it can be completely ignored? It can be used
without macro guards.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from dominiq at lps dot ens dot fr 2010-09-08 14:46 ---
Here is a better patch: ...
Yes! it accepts
program main
type b_obj
integer,allocatable :: c(:)
real :: r = 5.
end type b_obj
type (b_obj),allocatable :: b(:)
integer,allocatable :: c(:)
integer :: i,n
n =
--- Comment #13 from jakub at gcc dot gnu dot org 2010-09-08 14:48 ---
And, getdelim insist on allocating resp. reallocating the buffer. That is of
course usually desirable when used from C, but I'm afraid it isn't in this
case, where you have user provided buffer with fixed size. You
--- Comment #2 from tom dot browder at gmail dot com 2010-09-08 15:02
---
Ah, you are correct--old code may have that only. How about a warning instead
about using the recommended construct (the header guard) instead of the pragma
once?
--
--- Comment #3 from pinskia at gcc dot gnu dot org 2010-09-08 15:04 ---
At one point we deprecated it and then undeprecated it. See PR 11569.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45599
--- Comment #4 from tom dot browder at gmail dot com 2010-09-08 15:29
---
Oops, I missed that PR. I still think that an optional warning should be
there--something like -Wpragma-once with a message about the better practice.
(Sorry I missed finding the original bug--I only looked at
--- Comment #2 from hjl dot tools at gmail dot com 2010-09-08 15:30 ---
On Linux/x86-64, revision 163997 failed to build 191.fma3d in
SPEC CPU 2K:
[...@gnu-27 0001]$ /export/gnu/import/svn/gcc-test/usr/bin/gfortran -c -o
getirv.o -DSPEC_CPU2000_LP64 -O3 -funroll-loops
--- Comment #10 from sfilippone at uniroma2 dot it 2010-09-08 15:35 ---
(In reply to comment #9)
I have found a cure.
The original configuration had GMP, MPFR and MPC built and installed under the
user home directory (there were neither mpfr nor mpc system-wide, and gmp was a
bit
--- Comment #7 from dominiq at lps dot ens dot fr 2010-09-08 15:52 ---
The following code reduced from
http://groups.google.com/group/comp.lang.fortran/browse_thread/thread/76f99e7fd4f3e772#
module type2_type
implicit none
type, abstract :: Type2
character :: typeName*(30) =
--- Comment #11 from mikpe at it dot uu dot se 2010-09-08 16:00 ---
(In reply to comment #10)
(In reply to comment #9)
I have found a cure.
The original configuration had GMP, MPFR and MPC built and installed under the
user home directory (there were neither mpfr nor mpc
For the attached testcase, gcc generates a vmovapd for the store to
llvm_cbe__24__StackDv_P53. The latest Intel sde generates an alignment error:
SDE ERROR: ALIGN32 FAILED PC=40048b MEMEA=7ff057d0 vmovapd ymmword ptr
[rax], ymm0
It looks like gcc is considering 16-byte aligned data to be
--- Comment #1 from greened at obbligato dot org 2010-09-08 16:08 ---
Compile with -c -mavx reduced.c.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45600
--- Comment #2 from greened at obbligato dot org 2010-09-08 16:09 ---
Created an attachment (id=21740)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21740action=view)
Reduced testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45600
--- Comment #12 from sfilippone at uniroma2 dot it 2010-09-08 16:11 ---
(In reply to comment #11)
(In reply to comment #10)
How did you configure those prebuilt gmp/mpfr/mpc libraries installed under
your home directory? In particular, did you configure them all with
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-08 16:19 ---
Yes, please, and assign to me (working on a simplify_comparison fix for that).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20517
--- Comment #3 from hjl dot tools at gmail dot com 2010-09-08 16:19 ---
This is caused by revision 163973:
http://gcc.gnu.org/ml/gcc-cvs/2010-09/msg00265.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45598
--- Comment #1 from ibolton at gcc dot gnu dot org 2010-09-08 16:21 ---
reg is assigned to a temporary (reg.0) at the very first tree pass, as shown by
this 004t.gimple dump:
d ()
{
struct b * const reg.0;
unsigned int * D.2019;
int D.2020;
goto D.1276;
D.1275:
c ();
--- Comment #24 from rwild at gcc dot gnu dot org 2010-09-08 16:26 ---
(In reply to comment #22)
I've just read this thread and am now unsure as to whether there is a bug with
GCC or not. It sounds like your initial troubles have gone away and now there
is a new problem. Would it
The explicit template instantiation fails. As far as I can find on the net, it
should work. See attached minimal test case.
--
Summary: explicit template instanciation problem
Product: gcc
Version: unknown
Status: UNCONFIRMED
Severity:
--- Comment #1 from stephane at magnenat dot net 2010-09-08 16:30 ---
Created an attachment (id=21741)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21741action=view)
Minimal test case
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45601
--- Comment #2 from stephane at magnenat dot net 2010-09-08 16:31 ---
Output of g++ -v -save-temps -o test.o -c test.cpp :
Using built-in specs.
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.4.3-4ubuntu5'
--- Comment #3 from paolo dot carlini at oracle dot com 2010-09-08 16:44
---
Please clarify: As far as I can find on the net, it should work. No compiler
to which I have access compiles it, I tried, besides GCC, Intel, SunStudio,
Comeau, VC++8. Note I didn't really analyze the testcase
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-08 16:46 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 16:46:13 2010
New Revision: 164004
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164004
Log:
PR fortran/45595
* openmp.c (resolve_omp_do):
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-08 16:47 ---
Subject: Bug 45597
Author: jakub
Date: Wed Sep 8 16:47:16 2010
New Revision: 164005
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164005
Log:
PR fortran/45597
* trans-openmp.c
--- Comment #4 from paolo dot carlini at oracle dot com 2010-09-08 16:58
---
Actually, it seems pretty straightforward to me that S is nondeduced in the
last case: see 14.8.2.4/4, the last line.
--
paolo dot carlini at oracle dot com changed:
What|Removed
--- Comment #14 from jamborm at gcc dot gnu dot org 2010-09-08 17:01
---
Patches submitted to the mailing list for approval/comments:
http://gcc.gnu.org/ml/gcc-patches/2010-09/msg00674.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44972
--- Comment #28 from mikael at gcc dot gnu dot org 2010-09-08 17:21 ---
(In reply to comment #27)
What is the fate of the patch in comment #19?
Some (admittedly small) parts of the patch are already on trunk.
The transpose patches still waiting review at
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-08 17:22 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 17:22:36 2010
New Revision: 164007
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164007
Log:
PR fortran/45595
* openmp.c (resolve_omp_do):
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-08 17:24 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 17:23:52 2010
New Revision: 164008
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164008
Log:
PR fortran/45595
* openmp.c (resolve_omp_do):
--- Comment #2 from jamborm at gcc dot gnu dot org 2010-09-08 17:27 ---
Subject: Bug 45443
Author: jamborm
Date: Wed Sep 8 17:27:09 2010
New Revision: 164009
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164009
Log:
2010-09-08 Martin Jambor mjam...@suse.cz
PR
--- Comment #3 from jamborm at gcc dot gnu dot org 2010-09-08 17:36 ---
Subject: Bug 45443
Author: jamborm
Date: Wed Sep 8 17:36:40 2010
New Revision: 164011
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164011
Log:
2010-09-08 Martin Jambor mjam...@suse.cz
PR
--- Comment #3 from pinskia at gcc dot gnu dot org 2010-09-08 17:39 ---
I think this code is undefined with respect of alignment requirements.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45600
--- Comment #4 from pinskia at gcc dot gnu dot org 2010-09-08 17:43 ---
Yes this is invalid with respect of alignment requirements.
It becomes obvious from the optimized at -O0 on the trunk.
v4df llvm_cbe_r5585;
v4df llvm_cbe_r5584;
struct l_DV1 llvm_cbe__24__StackDv_P53;
--- Comment #9 from vmakarov at redhat dot com 2010-09-08 17:44 ---
The problem is in that pseudos (r121 in our case) spilled by IRA are
not added to live_throughout of reload chain. As the result,
pseudo_forbidden_regs are not set up for such pseudos and they can get
a hard registers
--- Comment #7 from nicola at gcc dot gnu dot org 2010-09-08 17:48 ---
Confirmed.
Thanks
--
nicola at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from nicola at gcc dot gnu dot org 2010-09-08 17:49 ---
Created an attachment (id=21742)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21742action=view)
A tidied up testcase ready for the GCC testsuite
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23215
--- Comment #8 from dominiq at lps dot ens dot fr 2010-09-08 17:49 ---
One of the first thing taught in fortran is that the loop ordering in the test
in comment #0 is bad.
If the loops are interchanged (as they should) the code vectorize. Is not
-floop-interchange supposed to do that
--- Comment #7 from nicola at gcc dot gnu dot org 2010-09-08 17:50 ---
Apologies, bugzilla confused me by jumping to the next bug and I attached a
testcase to the wrong bug! ;-)
Thanks
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23215
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-08 17:52 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #8 from nicola at gcc dot gnu dot org 2010-09-08 17:52 ---
Created an attachment (id=21743)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21743action=view)
A tidied up testcase ready for the GCC testsuite
--
nicola at gcc dot gnu dot org changed:
What
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-08 17:52 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
When a while loop with the true condition (while (1)) contains
an if statement with a modulus evaluated on a counter variable, it
appears that gcc incorrectly optimizes out the modulus check as a
constant (even though the counter variable is updated after the if
statement). If the counter
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-01-01 01:25 ---
That is because it is the same as PR 31217.
*** This bug has been marked as a duplicate of 31217 ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
1 - 100 of 159 matches
Mail list logo