--- Comment #16 from jvdelisle at gcc dot gnu dot org 2008-01-06 18:40
---
Fixed on trunk.
--
jvdelisle at gcc dot gnu dot org changed:
What|Removed |Added
S
--- Comment #15 from jvdelisle at gcc dot gnu dot org 2008-01-06 18:19
---
Subject: Bug 34387
Author: jvdelisle
Date: Sun Jan 6 18:19:06 2008
New Revision: 131356
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131356
Log:
2008-01-06 Jerry DeLisle <[EMAIL PROTECTED]>
--- Comment #14 from burnus at gcc dot gnu dot org 2008-01-06 09:39 ---
Posted patch for this PR and PR 34540.
http://gcc.gnu.org/ml/gcc-patches/2008-01/msg00192.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34387
--- Comment #13 from jvdelisle at gcc dot gnu dot org 2008-01-04 23:59
---
Great! I will work on the bound as well. I am also studying 34540 again in
light of this PR. These two are related.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34387
--- Comment #12 from rsandifo at nildram dot co dot uk 2008-01-04 10:15
---
Subject: Re: FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument
missmatch
"jvdelisle at gcc dot gnu dot org" <[EMAIL PROTECTED]> writes:
> Can you try this patch and see if it works and also send me
--- Comment #11 from burnus at gcc dot gnu dot org 2008-01-04 09:34 ---
(In reply to comment #10)
I like the patch, which creates (on x86-64/-m32|-m64) the dump:
_gfortran_cshift0_8 (&parm.0, &parm.2, &C.877, dimmy);
Can we do something about bound? It works but the conditional
expr
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2008-01-04 05:00
---
Notice in the tree dump that dimmy is already a pointer type when passed into
the subroutine.
Can you try this patch and see if it works and also send me the fdump tree
original?
Index: trans-expr.c
===
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2008-01-04 01:47
---
I will have a re-go at this again.
--
jvdelisle at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #8 from burnus at gcc dot gnu dot org 2008-01-03 16:43 ---
(In reply to comment #6)
> The problem is that cshift0's target-independent function type says that the
> SHIFT and DIM arguments are integers, rather than pointers to integers.
Indeed. Thanks Richard! The dump has: