--- Comment #4 from smspillaz at gmail dot com 2010-07-08 06:36 ---
(In reply to comment #3)
> Really I think this is a violation of the One definition rule.
>
What do you mean by this? Is it our bug (do we need to change RTLD_LAZY)? Or
GCC's bug?
If it is the former, can the document
--- Comment #6 from jv244 at cam dot ac dot uk 2010-07-08 06:24 ---
I have also tried to run the testcase with
export OMP_WAIT_POLICY=active
export GOMP_SPINCOUNT=infinity
which I would assume to keep the threads alive, and so there would be no need
to create these new threads (the nu
--- Comment #4 from tkoenig at gcc dot gnu dot org 2010-07-08 05:53 ---
Fixed on trunk. Closing.
--
tkoenig at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from bergner at gcc dot gnu dot org 2010-07-08 04:19 ---
The fix in Comment #8 fixes the test case on systems that have a default of
unsigned char (eg, powerpc*-linux).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44828
--- Comment #8 from bergner at gcc dot gnu dot org 2010-07-08 04:12 ---
Subject: Bug 44828
Author: bergner
Date: Thu Jul 8 04:12:04 2010
New Revision: 161942
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161942
Log:
PR middle-end/44828
* gcc.c-torture/execute/p
--- Comment #1 from moonshine at kapsi dot fi 2010-07-08 03:46 ---
Created an attachment (id=21137)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21137&action=view)
a preprocessed testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44865
Remove any of the options (or change -O3 to -O2) and this will pass. Found
while compiling Ghostscript.
gcc-trunk -c -O3 -march=pentium4 -flto ttinterp.i
In file included from ./base/ttobjs.h:42:0,
from ./base/ttinterp.h:53,
from ./base/ttinterp.c:68:
./base/setjm
--- Comment #2 from eli dot friedman at gmail dot com 2010-07-08 03:08
---
I'm afraid I mis-reduced the issue; try the following (which reproduces on both
trunk r161941 and 4.5.0):
typedef struct LLSModel{
double variance[32];
int indep_count;
}LLSModel;
void av_solve_lls(LLSM
This code generates an internal compiler error (the same code as in Bug Report
44863 but with certain lines uncommented):
!
module BaseStrategy
type, public, abstract :: Strategy
contains
procedure(strate
--- Comment #1 from david dot car7 at gmail dot com 2010-07-08 02:11
---
Created an attachment (id=21136)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21136&action=view)
test case described in post
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44863
The following code compiles fine but produces a runtime error when trying to
perform a dynamic dispatch:
!
module BaseStrategy
type, public, abstract :: Strategy
contains
procedure(strategy_update), pass( this ), deferred :
--- Comment #4 from moonshine at kapsi dot fi 2010-07-08 02:02 ---
(In reply to comment #3)
Bug mentioned in #3 was fixed on the mainline, so now this issue manifests
itself on trunk also. The difficulty is that a complete testcase would be to
build xz, the smaller case I added is unfor
--- Comment #34 from amylaar at gcc dot gnu dot org 2010-07-08 01:10
---
FWIW, this simple patch stops the comparison failures both for my reduced
testcase as for the original testcase. I'm not sure if we want to
pay the price of carryinig more labels around for -g0, or if we'd rather
--- Comment #33 from amylaar at gcc dot gnu dot org 2010-07-08 01:00
---
pop_labels_1 installs the LABEL_DECL of half in BLOCK_VARS:
Breakpoint 16, pop_labels_1 (slot=0xb7d86f94, data=0xb7d95068)
at ../../gcc/gcc/cp/decl.c:384
384 struct named_label_entry *ent = (struct named
--- Comment #3 from howarth at nitro dot med dot uc dot edu 2010-07-08
00:40 ---
Created an attachment (id=21135)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21135&action=view)
diff between stage2 build of libcpp without and with --enable-build-with-cxx
--
http://gcc.gnu.or
-20100707/libcpp -I. -I../../gcc-4.6-20100707/libcpp/../include
-I../../gcc-4.6-20100707/libcpp/include -I/sw/include -g -O2 -W -Wall
-Wwrite-strings -Wmissing-format-attribute -Wstrict-prototypes
-Wmissing-prototypes -Wold-style-definition -Wc++-compat -pedantic
-Wno-long-long -Werror -I../../gcc-4.6
--- Comment #1 from howarth at nitro dot med dot uc dot edu 2010-07-08
00:09 ---
Created an attachment (id=21134)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21134&action=view)
compressed build log on x86_64-apple-darwin10
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4486
-4.6.0-1000/gcc-4.6-20100707/libstdc++-v3/libsupc++
-L/sw/src/fink.build/gcc46-4.6.0-1000/darwin_objdir/prev-x86_64-apple-darwin10.4.0/libstdc++-v3/src/.libs
-I../../gcc-4.6-20100707/libcpp -I. -I../../gcc-4.6-20100707/libcpp/../include
-I../../gcc-4.6-20100707/libcpp/include -g -O2 -gtoggle -W
--- Comment #11 from sje at cup dot hp dot com 2010-07-07 23:45 ---
Created an attachment (id=21133)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21133&action=view)
Compressed decl.c.043t.inline_param3 file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44716
--- Comment #10 from sje at cup dot hp dot com 2010-07-07 23:45 ---
Created an attachment (id=21132)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21132&action=view)
Compressed decl.c.041t.fnsplit file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44716
--- Comment #9 from sje at cup dot hp dot com 2010-07-07 23:44 ---
Created an attachment (id=21131)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21131&action=view)
Compressed decl.c.025t.einline2 file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44716
--- Comment #8 from sje at cup dot hp dot com 2010-07-07 23:44 ---
Created an attachment (id=21130)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21130&action=view)
Compressed decl.c.015t.inline_param1 file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44716
--- Comment #7 from sje at cup dot hp dot com 2010-07-07 23:43 ---
Created an attachment (id=21129)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21129&action=view)
Compressed preprocessed cp/decl.c
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44716
--- Comment #6 from hubicka at ucw dot cz 2010-07-07 23:01 ---
Subject: Re: [4.6 Regression] Bootstrap fails with
partial inlining (r161382)
great work! Could you, please, post me the fnsplit, einline2 and inline dumps?
Perhaps I can work out what is wrong there.
(and preproce
--- Comment #7 from bergner at gcc dot gnu dot org 2010-07-07 22:51 ---
Jakub's test case still fails on powerpc*-linux because we default to unsigned
char. I think the obvious fix is to just pass -fsigned-char in dg-options.
--
bergner at gcc dot gnu dot org changed:
Wh
--- Comment #5 from sje at cup dot hp dot com 2010-07-07 22:48 ---
If I put __attribute__ ((noinline)) on check_class_member_definition_namespace
in cp/decl.c, I don't see the bug. I don't see anything special about this
function so I don't know why it is having problems being (partiall
--- Comment #8 from hjl dot tools at gmail dot com 2010-07-07 21:42 ---
Fixed.
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Status|UNCONFIRM
--- Comment #4 from hjl dot tools at gmail dot com 2010-07-07 21:42 ---
Fixed.
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Status|UNCONFIRM
--- Comment #7 from hjl dot tools at gmail dot com 2010-07-07 21:41 ---
Fixed.
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Status|UNCONFIRM
--- Comment #3 from hjl at gcc dot gnu dot org 2010-07-07 21:39 ---
Subject: Bug 44844
Author: hjl
Date: Wed Jul 7 21:39:30 2010
New Revision: 161936
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161936
Log:
Backport support for AVX Programming Reference (June, 2010).
gcc/
2
--- Comment #2 from burnus at gcc dot gnu dot org 2010-07-07 21:28 ---
(In reply to comment #1)
> I won't be able to start a regression hunt until this week-end.
Working: 2010-04-10-r158177
Failing: 2010-04-13-r158277
The array constructor work was merged as Rev. 158253 - thus, as writ
--- Comment #7 from ebotcazou at gcc dot gnu dot org 2010-07-07 21:24
---
> Fixed in commits 161918 (mainline) and 161919 (gcc-4.5 branch).
The right procedure is to copy-and-paste the ChangeLog entry in the commit log,
this will automatically add a cross-reference with an URL to this
--- Comment #7 from hjl at gcc dot gnu dot org 2010-07-07 21:11 ---
Subject: Bug 44588
Author: hjl
Date: Wed Jul 7 21:11:25 2010
New Revision: 161933
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161933
Log:
Backport 8bit div/mod improvements.
gcc/
2010-07-07 H.J. Lu
--- Comment #12 from hjl at gcc dot gnu dot org 2010-07-07 21:11 ---
Subject: Bug 44695
Author: hjl
Date: Wed Jul 7 21:11:25 2010
New Revision: 161933
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161933
Log:
Backport 8bit div/mod improvements.
gcc/
2010-07-07 H.J. Lu
> cat bug.ii
bool f();
struct counted_base {
virtual void destroy() { }
void release() { if (f()) destroy(); }
};
struct shared_count {
shared_count() { }
~shared_count() { if (pi) pi->release(); }
shared_count(shared_count& r) : pi(r.pi) { if (pi) pi->rele
--- Comment #32 from amylaar at gcc dot gnu dot org 2010-07-07 19:53
---
tree-vrp.c:find_switch_asserts uses compare_case_labels to sort case labels,
with the main key being the uid of the case label.
We have different case labels when debugging, and their uid sorts differently
in relat
--
rth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |rth at gcc dot gnu dot org
|dot org
--- Comment #1 from raj dot khem at gmail dot com 2010-07-07 19:35 ---
Created an attachment (id=21128)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21128&action=view)
testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44860
The attached program causes spill failure on latest 4.5 branch with -mthumb
it fails when no -O is used and works when -Os/-O2 is used but fails again
when -fno-omit-frame-pointer is passed in addition to -Os or -O2
arm-oe-linux-uclibceabi-gcc -mthumb -Os -fno-omit-frame-pointer recvfrom.i
recv
--- Comment #19 from changpeng dot fang at amd dot com 2010-07-07 19:00
---
(In reply to comment #18)
> Changpeng, should this PR be closed now?
>
No. I am still looking at the dependence computation cost. I just found the
most of the time is spent in memory allocation and freeing of
--- Comment #18 from spop at gcc dot gnu dot org 2010-07-07 18:14 ---
Changpeng, should this PR be closed now?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44576
--- Comment #31 from amylaar at gcc dot gnu dot org 2010-07-07 17:50
---
(In reply to comment #30)
The first difference I seen in debug / no debug values of
cfun->gimple_df->free_ssanames->ssa_name.version for the relevant function
is in vrp.
Debug:
Hardware watchpoint 10: cfun->gimpl
--- Comment #30 from amylaar at gcc dot gnu dot org 2010-07-07 17:27
---
(In reply to comment #28)
> No, the culprit is loop header copying which causes
I also see changes in the MEM_* after loop header copying.
It happens during the second call of copy_loop_header. Alas,
it seems tha
--- Comment #4 from sje at cup dot hp dot com 2010-07-07 17:22 ---
The problem seems to happen when compiling cp/decl.c. If I compile this file
at -O1 instead of -O2 the resulting C++ compiler will work. I am trying to see
if I can track it down to one function within cp/decl.c.
--
--- Comment #3 from hp at gcc dot gnu dot org 2010-07-07 16:24 ---
(In reply to comment #2)
> The addition by 10 appears during postreload, where we substitute
> ...
> which by itself doesn't look wrong.
That's right, if the two stores had been to different locations, it'd have been
opt
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-07-07 16:24
---
I won't be able to start a regression hunt until this week-end.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44857
--- Comment #2 from ro at gcc dot gnu dot org 2010-07-07 15:54 ---
A reghunt identified this patch as the culprit:
2010-07-05 Richard Guenther
* tree.c (reference_alias_ptr_type): New function.
* tree.h (reference_alias_ptr_type): Declare.
* tree-ssa-loop-ivo
--- Comment #1 from ro at gcc dot gnu dot org 2010-07-07 15:54 ---
Created an attachment (id=21127)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21127&action=view)
preprocessed libfortran/io/format.c
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44843
--- Comment #3 from sje at cup dot hp dot com 2010-07-07 15:30 ---
I haven't been able to come up with a test case other then bootstrapping. If I
build a non-bootstrap compiler and run the testsuite I don't get any unexpected
failures due to this problem. It is only when, during bootst
--- Comment #8 from bernds at gcc dot gnu dot org 2010-07-07 15:27 ---
Subject: Bug 44404
Author: bernds
Date: Wed Jul 7 15:26:48 2010
New Revision: 161920
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161920
Log:
PR rtl-optimization/44404
* auto-inc-dec.c (fin
--- Comment #6 from baldrick at gcc dot gnu dot org 2010-07-07 15:25
---
Fixed in commits 161918 (mainline) and 161919 (gcc-4.5 branch).
--
baldrick at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #3 from burnus at gcc dot gnu dot org 2010-07-07 15:24 ---
See PR 44856 for the reason why it fails for some expressions.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40571
--- Comment #2 from burnus at gcc dot gnu dot org 2010-07-07 15:23 ---
Note: The tricky part for a simple fix is:
moduleArray(i)
If one simply wraps in resolve_code the code->expr(1,2) with parentheses (e =
gfc_get_parentheses (e)") this leads to "D.123 = modulearray[i]", which does
--- Comment #2 from burnus at gcc dot gnu dot org 2010-07-07 15:20 ---
Created an attachment (id=21126)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21126&action=view)
Test case
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40571
--- Comment #29 from amylaar at gcc dot gnu dot org 2010-07-07 15:06
---
(In reply to comment #26)
> thus I think we have to look for an instability in the SCC walker.
There's a qsort call. Is compare_ops guaranteed never to return 0 for
items that are not identical?
--
http://gc
--
redi at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Keyword
--- Comment #28 from rguenth at gcc dot gnu dot org 2010-07-07 15:00
---
No, the culprit is loop header copying which causes
@@ -2634,8 +2634,8 @@
goto ;
:
- # .MEM_6 = PHI <.MEM_106(7), .MEM_105(D)(6)>
- # .MEM_107 = VDEF <.MEM_6>
+ # .MEM_149 = PHI <.MEM_106(7), .MEM_105(D)
--- Comment #27 from amylaar at gcc dot gnu dot org 2010-07-07 14:56
---
(In reply to comment #26)
> The first difference appears in .093t.pre (comparing -nouid dumps):
>
> cat ii386.3.3.i.gk.094t.pre | grep -v '# DEBUG' | diff -u ii386.3.3.i.094t.pre
> - | less
is that .093t.pre or
struct Base1 {};
struct Derived1 : public Base1 {};
const Base1& f1() { Base1 x; return x; }
const Base1& f2() { return Base1(); }
const Base1& f3() { Derived1 x; return x; }
const Base1& f4() { return Derived1(); }
struct Base2 { int m_foo; };
struct Derived2 : public Base2 {};
const Base2& f5(
--- Comment #26 from rguenth at gcc dot gnu dot org 2010-07-07 14:48
---
The first difference appears in .093t.pre (comparing -nouid dumps):
cat ii386.3.3.i.gk.094t.pre | grep -v '# DEBUG' | diff -u ii386.3.3.i.094t.pre
- | less
void
_ZL31ix86_expand_vector_init_generalb12machine_mo
--- Comment #25 from amylaar at gcc dot gnu dot org 2010-07-07 14:47
---
Created an attachment (id=21125)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21125&action=view)
diff -u -I '^ # DEBUG.*$' t.c.093t.pre t.c.gk.093t.pre
Disregarding where merely the base names of ssa name
--- Comment #1 from burnus at gcc dot gnu dot org 2010-07-07 14:47 ---
My idea is:
a) For scalar expressions, generate simply always the scalar variable.
b) For array accesses, generate a const variable at the beginning of the
current block (subroutine, function, BLOCK) - if the variab
--- Comment #1 from dmitrij dot ledkov at ubuntu dot com 2010-07-07 14:39
---
Duplicate of Bug #40868.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42311
--- Comment #24 from rguenth at gcc dot gnu dot org 2010-07-07 14:34
---
Created an attachment (id=21124)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21124&action=view)
less reduced testcase
Reduced at topformflat level 0 only.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?i
--- Comment #23 from jakub at gcc dot gnu dot org 2010-07-07 14:25 ---
Hash tables hashed by DECL_UID should be traversed only when the traversal
order doesn't affect code generation. E.g. Richard fixed a few places to just
set
bits in a bitmap and then iterate over the bitmap for code
--- Comment #27 from sfilippone at uniroma2 dot it 2010-07-07 14:25 ---
(In reply to comment #26)
For the test_coo.f03 case, what happens is shown in the following dump.
{
struct class$base_sparse_mat class.6;
if (vtab$d_coo_sparse_mat.allocate == 0B)
{
vtab$d_co
reg...@john-home:~$ current-gcc -O0 small.c -o small
reg...@john-home:~$ ./small
checksum g_610 = 1
reg...@john-home:~$ current-gcc -O1 small.c -o small
reg...@john-home:~$ ./small
checksum g_610 = 0
reg...@john-home:~$ current-gcc -v
Using built-in specs.
COLLECT_GCC=current-gcc
COLLECT_LTO_WRAPPE
--- Comment #22 from amylaar at gcc dot gnu dot org 2010-07-07 14:06
---
(In reply to comment #18)
> That's not SSA name, but DECL_UID of the underlying decl.
> I believe such changes are considered ok if it just means different gaps
> between uids with -g vs. -g0.
For compiler-generat
--- Comment #21 from amylaar at gcc dot gnu dot org 2010-07-07 14:02
---
(In reply to comment #19)
> Yes, that's ok. I'm re-reducing the testcase (which also fails with
> -fno-ivopts).
FWIW I've reduced the testcase with the aim of reproducing the problem in
r160832, which is when the
--- Comment #20 from amylaar at gcc dot gnu dot org 2010-07-07 14:00
---
Created an attachment (id=21123)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21123&action=view)
typescript looking at inrements of next_decl_uid
The last ssa name before 1881 in the dump files is 1872; so
--- Comment #30 from rguenth at gcc dot gnu dot org 2010-07-07 13:58
---
I'm going to test the patch in comment #28.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #19 from rguenth at gcc dot gnu dot org 2010-07-07 13:57
---
Yes, that's ok. I'm re-reducing the testcase (which also fails with
-fno-ivopts).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44832
--- Comment #8 from nathan at gcc dot gnu dot org 2010-07-07 13:56 ---
Hm, I guess I must have flubbed the testcase. After all this time, I don't
have a better recollection. Sorry.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43120
--- Comment #18 from jakub at gcc dot gnu dot org 2010-07-07 13:55 ---
That's not SSA name, but DECL_UID of the underlying decl.
I believe such changes are considered ok if it just means different gaps
between uids with -g vs. -g0.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4483
--- Comment #17 from amylaar at gcc dot gnu dot org 2010-07-07 13:48
---
(In reply to comment #16)
> Maybe I'm blind, but the SSA name versions are all the same in
> the above diff.
The ssa name 1881 gets changed to 1882, isn't that significant?
--
http://gcc.gnu.org/bugzilla/show
--
burnus at gcc dot gnu dot org changed:
What|Removed |Added
Known to fail||4.6.0
Known to work||4.5.0
Target
Juan Rodriguez-Carvajal reported at
http://gcc.gnu.org/ml/fortran/2010-07/msg00057.html
that compiling the following program ICEs with:
f951.exe: internal compiler error: in output_constructor_regular_field, at
varasm.c:4996
It works with GCC 4.5 and might be due to the constructor reorganizatio
--- Comment #6 from hjl at gcc dot gnu dot org 2010-07-07 13:36 ---
Subject: Bug 44850
Author: hjl
Date: Wed Jul 7 13:36:31 2010
New Revision: 161911
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161911
Log:
Revert revision 161876.
2010-07-07 H.J. Lu
PR target/448
--- Comment #2 from hjl at gcc dot gnu dot org 2010-07-07 13:33 ---
Subject: Bug 44844
Author: hjl
Date: Wed Jul 7 13:33:04 2010
New Revision: 161910
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161910
Log:
Retry rdrand if the carry flag isn't valid.
gcc/
2010-07-07 H.J. L
--- Comment #16 from rguenther at suse dot de 2010-07-07 13:24 ---
Subject: Re: [4.6 Regression] -fcompare-debug failure for
C++ i386.c
On Wed, 7 Jul 2010, amylaar at gcc dot gnu dot org wrote:
> --- Comment #14 from amylaar at gcc dot gnu dot org 2010-07-07 13:22
> ---
> (
--- Comment #7 from jason at gcc dot gnu dot org 2010-07-07 13:24 ---
That thread does help, thanks. But part of my confusion was that the testcase
you added with that patch (abi/covariant1) didn't actually seem to test the
bug. When I add a definition of c14::f17, the test starts fail
--- Comment #15 from jakub at gcc dot gnu dot org 2010-07-07 13:23 ---
The reduced testcase doesn't fail for me, the unreduced one fails though.
-fcompare-debug complains about different ORIGINAL_REGNOs and different ivtmp.*
names.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4483
--- Comment #14 from amylaar at gcc dot gnu dot org 2010-07-07 13:22
---
(In reply to comment #12)
$ diff -u -I '^ # DEBUG.*$' t.c.025t.* t.c.gk.025t.*
--- t.c.025t.einline2 2010-07-07 13:59:11.251978485 +0100
+++ t.c.gk.025t.einline22010-07-07 13:59:11.451101846 +0100
@@ -14
--- Comment #13 from amylaar at gcc dot gnu dot org 2010-07-07 13:15
---
(In reply to comment #12)
> Hm, different SSA name versions are not good - that might cause
> code generation differences. Where do they first differ?
t.c.025t.einline2 / t.c.gk.025t.einline2
--
http://gcc.
--- Comment #26 from sfilippone at uniroma2 dot it 2010-07-07 13:15 ---
(In reply to comment #25)
This fixes test_vt2 but not test_coo. Will keep investigating.(and of
course it should be done for functions as well as subroutines...)
Index: gcc/fortran/resolve.c
===
--- Comment #3 from domob at gcc dot gnu dot org 2010-07-07 13:05 ---
See http://gcc.gnu.org/ml/fortran/2010-07/msg00058.html.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44709
--- Comment #7 from rguenth at gcc dot gnu dot org 2010-07-07 13:04 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRM
--- Comment #6 from rguenth at gcc dot gnu dot org 2010-07-07 12:43 ---
Subject: Bug 44790
Author: rguenth
Date: Wed Jul 7 12:43:38 2010
New Revision: 161907
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161907
Log:
2010-07-07 Richard Guenther
PR middle-end/44790
Currently, gfortran saves scalar constants (PARAMETER) only in the .mod file
while array parameters are saved as global, static variable in the .o file and
additionally as expression in the .mod file.
This has all kind of funny consequences. For instance:
print *, array_par, (array_par), shape
--- Comment #29 from matz at gcc dot gnu dot org 2010-07-07 12:10 ---
[just for completeness to not lose the thought:]
Thinking about this some more (triggered by the problem of not having nice
back edges in irreducible loops), it's not really the back edges that are
interesting but the
--- Comment #28 from rguenth at gcc dot gnu dot org 2010-07-07 11:59
---
The following is a fix (or workaround) for the problem.
Index: gcc/tree-ssa-alias.c
===
--- gcc/tree-ssa-alias.c(revision 161869)
+++ gcc/tre
--- Comment #27 from rakdver at kam dot mff dot cuni dot cz 2010-07-07
11:31 ---
Subject: Re: [4.6 regression] RTL loop
unrolling causes FAIL: gcc.dg/pr39794.c
> Due to SSA form the alias information reflects dependencies only between
> accesses as if it ignores back edges.
W
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-07-07 11:22 ---
The addition by 10 appears during postreload, where we substitute
(insn 24 35 65 4 t.i:9 (set (reg:QI 9 r9 [orig:44 prephitmp.3 ] [44])
(const_int 58 [0x3a])) 45 {movqi} (expr_list:REG_EQUAL (const_int 58
[0
--- Comment #26 from rakdver at kam dot mff dot cuni dot cz 2010-07-07
11:19 ---
Subject: Re: [4.6 regression] RTL loop
unrolling causes FAIL: gcc.dg/pr39794.c
> In
>
>...
>*p_1 = x;
>y = *(p_1 + 1);
>...
>
> I can say that *p_1 does not alias *(p_1 + 1) inde
--- Comment #25 from matz at gcc dot gnu dot org 2010-07-07 11:15 ---
Due to SSA form the alias information reflects dependencies only between
accesses as if it ignores back edges. Hence any transformation that
transforms a back edge into a forward edge, or moves code over back edges
ne
--- Comment #24 from rguenth at gcc dot gnu dot org 2010-07-07 11:06
---
In
...
*p_1 = x;
y = *(p_1 + 1);
...
I can say that *p_1 does not alias *(p_1 + 1) independent on what code
is around. If it would be
BB3:
# p_1 = PHI
*p_1 = x;
y = *(p_1 + 1);
p_2 = p_1 +
--- Comment #23 from rakdver at kam dot mff dot cuni dot cz 2010-07-07
10:51 ---
Subject: Re: [4.6 regression] RTL loop
unrolling causes FAIL: gcc.dg/pr39794.c
> > > > > > > Consider this simplified example:
> > > > > > >
> > > > > > > for (i ...)
> > > > > > > {
> > > > >
--- Comment #22 from rguenth at gcc dot gnu dot org 2010-07-07 10:48
---
(In reply to comment #21)
> Subject: Re: [4.6 regression] RTL loop
> unrolling causes FAIL: gcc.dg/pr39794.c
>
> > > > > > Consider this simplified example:
> > > > > >
> > > > > > for (i ...)
> > > > >
--- Comment #21 from rakdver at kam dot mff dot cuni dot cz 2010-07-07
10:47 ---
Subject: Re: [4.6 regression] RTL loop
unrolling causes FAIL: gcc.dg/pr39794.c
> > > > > Consider this simplified example:
> > > > >
> > > > > for (i ...)
> > > > > {
> > > > > /*A*/ t = a[i];
--- Comment #20 from rguenth at gcc dot gnu dot org 2010-07-07 10:43
---
(In reply to comment #19)
> Subject: Re: [4.6 regression] RTL loop
> unrolling causes FAIL: gcc.dg/pr39794.c
>
> > > > > I am not sure what you mean -- I may be misunderstanding how rtl
> > > > > alias a
1 - 100 of 125 matches
Mail list logo
