--- Comment #1 from janus at gcc dot gnu dot org 2010-08-02 06:56 ---
Probably related to PR 44584.
--
janus at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #35 from paolo dot carlini at oracle dot com 2010-08-02 06:53
---
Thanks a lot Armand (by the way, many thanks to Ralf too)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40974
--- Comment #10 from dominiq at lps dot ens dot fr 2010-08-02 05:23 ---
The patch in comment #9 works as expected on the gfortran testsuite and on my
own tests. Note that while testing I have found unrelated pr45161. Thanks for
the patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?
--- Comment #3 from jiez at gcc dot gnu dot org 2010-08-02 04:34 ---
Aggregates Copy Propagation should be able to fix this, too.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45144
When compiled with -std=f2003 gfortran.dg/typebound_proc_16.f03 yields an ICE:
/opt/gcc/work/gcc/testsuite/gfortran.dg/typebound_proc_16.f03:18.49:
PROCEDURE,PRIVATE :: rat_asgn_i, rat_plus_rat, rat_plus_i
1
Error: Fortran 2008: PROCEDURE list
--- Comment #5 from jsdeckerido at gmail dot com 2010-08-02 01:22 ---
how is this invalid?
it shouldn't matter whether map defines operator[](std::initializer_list<>),
only operator[](std::pair) for std::map,int>.
{1, 2} should then be used to initialize that pair.
--
http://gcc.g
--- Comment #5 from qiyao at gcc dot gnu dot org 2010-08-02 00:38 ---
(In reply to comment #4)
> > Post patch here http://gcc.gnu.org/ml/gcc-patches/2010-08/msg1.html
>
> + output_asm_insn ("strr%?\t%H0, [%1, #4]", otherops);
>
> s/strr/str/ ?
>
Right, it is a typo
--- Comment #2 from t dot artem at mailcity dot com 2010-08-02 00:31
---
Created an attachment (id=21368)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21368&action=view)
faad2 2.7.0 complete sources with -save-temps
Complete sources with -save-temps for GCC 4.2.4 and GCC 4.4.4.
--- Comment #1 from t dot artem at mailcity dot com 2010-08-02 00:09
---
faad2 2.7.0 exhibits the same misbehavior, you can download it here:
http://downloads.sourceforge.net/faac/faad2-2.7.tar.bz2
No extra commands are required for compilation, just ./configure && make
---
P.S. You
A testcase for x86 architecture (Intel Core i5 CPU here):
1) Download
http://sourceforge.net/projects/faac/files/faad2-src/faad2-2.6/faad2-2.6.1.tar.gz/download
2) Build it using these commands
perl -pi -e 's|dnl AC_PROG_CXX|AC_PROG_CXX|' configure.in
autoreconf -vif
configure --disable-static
--- Comment #9 from mikael at gcc dot gnu dot org 2010-08-01 22:29 ---
Created an attachment (id=21367)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21367&action=view)
Tentative patch
I will test & submit this tomorrow.
I have decided to use gfc_commit_symbol (sym) instead of t
--- Comment #8 from janus at gcc dot gnu dot org 2010-08-01 21:30 ---
> > FAIL: gfortran.dg/proc_ptr_result_2.f90 -O (internal compiler error)
This one is fixed by adding a 'gfc_commit_symbols' also in
'gfc_copy_formal_args_intr' (cf. comment #3).
Still Remaining :
> > FAIL: gfortra
--- Comment #13 from burnus at gcc dot gnu dot org 2010-08-01 19:38 ---
(In reply to comment #12)
> This appears to be fixed now from the commit of patch in #10.
Which is the commit:
Date: Sat Jul 31 21:37:25 2010
New Revision: 162788
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&
--- Comment #10 from janus at gcc dot gnu dot org 2010-08-01 19:24 ---
Fixed with r162804. Closing.
--
janus at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from janus at gcc dot gnu dot org 2010-08-01 19:22 ---
Subject: Bug 44912
Author: janus
Date: Sun Aug 1 19:21:49 2010
New Revision: 162804
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162804
Log:
2010-08-01 Janus Weil
PR fortran/44912
* clas
--- Comment #6 from arekm at pld-linux dot org 2010-08-01 18:33 ---
Is there a patch for 4.5.1 for this somewhere? I'm hitting this issue at gcc
4.5.1 java boostrap (#45154)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44914
--- Comment #4 from tkoenig at gcc dot gnu dot org 2010-08-01 18:12 ---
This piece of code
/* If no intention of reversing or reversing is explicitly
inhibited, convert backward dependence to overlap. */
if ((reverse == NULL && th
--- Comment #3 from tkoenig at gcc dot gnu dot org 2010-08-01 17:37 ---
Confirmed for the test cases in comment #0.
You were right that my patch at
http://gcc.gnu.org/ml/fortran/2010-08/msg3.html
doesn't fix this. I'll have a look...
--
tkoenig at gcc dot gnu dot org changed:
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-08-01 17:28
---
This appears to be fixed now from the commit of patch in #10. The related
failures showing in gcc test results are no longer there.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
--- Comment #2 from burnus at gcc dot gnu dot org 2010-08-01 17:28 ---
(In reply to comment #1)
> ! --- FOUR (also occurs for Octopus)
> c = matmul(b, c)
As pointed out by Dominique, one needs to be careful. I think one can optimize:
c = matmul(b, transpose(c))
c = ma
--- Comment #34 from armand dot potter at free dot fr 2010-08-01 17:21
---
Overall 3/ is not that good as in fact it will use directly system headers
without any wrapper (from in-tree or installed gcc).
I tried adding -nostdinc++ to PCHFLAGS and compilation runs OK (not that hard
cons
--- Comment #1 from burnus at gcc dot gnu dot org 2010-08-01 16:52 ---
! --- FOUR (also occurs for Octopus)
subroutine t2(b,c)
implicit none
REAL :: b(3,3),c(3)
c = matmul(b, c)
end subroutine
That's a pattern I see quite often: LHS variable on
Using today's trunk with http://gcc.gnu.org/ml/fortran/2010-08/msg3.html
applied.
The following example were extracted from FLEUR (www.flapw.de), where
-Warray-temporaries showed that a temporary has been generated.
!-- ONE --
subroutine t1(n1,n2, gfft, ufft)
--- Comment #8 from bergner at gcc dot gnu dot org 2010-08-01 15:23 ---
Answering Steven's question from Comment #7: Yes.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10474
--- Comment #2 from loufoque at gmail dot com 2010-08-01 13:41 ---
My bad, this is actually normal, the additional bytes are put in the state and
will be returned later.
--
loufoque at gmail dot com changed:
What|Removed |Added
--- Comment #7 from mikael at gcc dot gnu dot org 2010-08-01 13:12 ---
(In reply to comment #3)
> With this patch, the remaining failures are:
>
> FAIL: gfortran.dg/bind_c_dts_3.f03 -O (internal compiler error)
> FAIL: gfortran.dg/c_assoc_2.f03 -O (internal compiler error)
fixed by
The following code compiles without an error
g++ -c a-bug.cc -o a-bug.o
//-
struct X
{
struct Y
{
void method();
};
void method();
};
void X::X::X::X::method()
// more than one X
{
}
void X::X::Y::Y::Y::method()
// more than one X and Y
{
}
//
--- Comment #6 from mikael at gcc dot gnu dot org 2010-08-01 12:54 ---
(In reply to comment #4)
> Btw, maybe 'changed_syms' should be made static again. Instead of the assert
> in
> parse.c one can use 'gfc_symbol_state'.
>
Yes, makes sense.
--
http://gcc.gnu.org/bugzilla/show_bug
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-01 12:52 ---
(In reply to comment #3)
> Many of these failures are actually due to 'gfc_copy_formal_args' and can be
> fixed with:
[...]
>
>
Agreed. The patch I was about to post had also this extra hunk :
Index: symbol.c
=
--- Comment #47 from ubizjak at gmail dot com 2010-08-01 12:50 ---
(In reply to comment #39)
> or alpha could make the va_list struct copies volatile. Or we can
> schedule tree-stdarg earlier.
Well, following patch fixes remaining gcc.c-torture/execute/stdarg-1.c failure
as well:
In
--- Comment #4 from janus at gcc dot gnu dot org 2010-08-01 12:50 ---
Btw, maybe 'changed_syms' should be made static again. Instead of the assert in
parse.c one can use 'gfc_symbol_state'.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45151
--- Comment #3 from janus at gcc dot gnu dot org 2010-08-01 12:39 ---
Many of these failures are actually due to 'gfc_copy_formal_args' and can be
fixed with:
Index: gcc/fortran/symbol.c
===
--- gcc/fortran/symbol.c
--
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=45151
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-08-01 12:29 ---
Confirmed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
CC|
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-08-01 12:25 ---
This looks like the same as PR44914.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from mikael at gcc dot gnu dot org 2010-08-01 12:23 ---
Confirmed, and mine I guess.
I don't know what I was testing when I said I was testing the patch for the
future revision 162776.
Actually, I was a bit surprised not to see any failure in the testsuite.
Anyway, the pa
--- Comment #6 from ubizjak at gmail dot com 2010-08-01 11:41 ---
FYI, following shoot-in-the-dark patch didn't help, but I guess it is needed
together with some further reload surgery:
Index: gensupport.c
===
--- gensuppor
--- Comment #4 from akos dot pasztory at gmail dot com 2010-08-01 11:18
---
> Post patch here http://gcc.gnu.org/ml/gcc-patches/2010-08/msg1.html
+ output_asm_insn ("strr%?\t%H0, [%1, #4]", otherops);
s/strr/str/ ?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?i
--- Comment #9 from ubizjak at gmail dot com 2010-08-01 10:43 ---
Fixed.
--
ubizjak at gmail dot com changed:
What|Removed |Added
Status|NEW
--- Comment #8 from uros at gcc dot gnu dot org 2010-08-01 10:42 ---
Subject: Bug 45142
Author: uros
Date: Sun Aug 1 10:41:57 2010
New Revision: 162797
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162797
Log:
PR target/45142
* config/i386/sse.md (vec_set_0): D
--- Comment #7 from steven at gcc dot gnu dot org 2010-08-01 10:40 ---
Isn't this bug description just a request for shrink-wrapping?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10474
--- Comment #3 from steven at gcc dot gnu dot org 2010-08-01 10:38 ---
Confirmed.
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
CC|
--- Comment #7 from ubizjak at gmail dot com 2010-08-01 10:01 ---
(In reply to comment #6)
> - (set_attr "mode" "SF")])
> + (set_attr "mode" "SF,SF,*")])
>
> SI?
No, this belongs to movss insns from "vec_set_0" pattern.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45142
--- Comment #2 from pluto at agmk dot net 2010-08-01 09:57 ---
svn-bisect result:
r161604 | jamborm | 2010-06-30 15:26:17 +0200 (śro) | 15 linii
2010-06-30 Martin Jambor
PR tree-optimization/43905
--- Comment #1 from pluto at agmk dot net 2010-08-01 09:55 ---
Created an attachment (id=21366)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21366&action=view)
testcase
compile with -O1 -c -fipa-sra -fnon-call-exceptions natClip.ii
--
http://gcc.gnu.org/bugzilla/show_bug.cgi
--- Comment #8 from laurent at guerby dot net 2010-08-01 09:53 ---
As of r162787 bootstrap goes a bit further then fails on compare in
stage3-bubble:
Comparing stages 2 and 3
warning: gcc/cc1-checksum.o differs
Bootstrap comparison failure!
gcc/tree-vect-data-refs.o differs
make[2]: ***
A build log from Mozilla showing this:
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1280652762.1280653204.15224.gz
I can gather up the response files if necessary, but I would imagine the
problem is that gcc is feeding the expanded response files to collect2 and
killing it.
--
--- Comment #3 from qiyao at gcc dot gnu dot org 2010-08-01 08:44 ---
(In reply to comment #2)
> Created an attachment (id=21327)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21327&action=view) [edit]
> simplified testcase
Update this test case a little bit, with test commands.
P
--- Comment #18 from burnus at gcc dot gnu dot org 2010-08-01 07:25 ---
*** Bug 45155 has been marked as a duplicate of this bug. ***
--
burnus at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from burnus at gcc dot gnu dot org 2010-08-01 07:25 ---
*** This bug has been marked as a duplicate of 39427 ***
--
burnus at gcc dot gnu dot org changed:
What|Removed |Added
50 matches
Mail list logo