[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-06 Thread jvdelisle at gcc dot gnu dot org
--- 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

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-06 Thread jvdelisle at gcc dot gnu dot org
--- 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]>

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-06 Thread burnus at gcc dot gnu dot org
--- 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

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-04 Thread jvdelisle at gcc dot gnu dot org
--- 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

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-04 Thread rsandifo at nildram dot co dot uk
--- 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

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-04 Thread burnus at gcc dot gnu dot org
--- 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

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-03 Thread jvdelisle at gcc dot gnu dot org
--- 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 ===

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-03 Thread jvdelisle at gcc dot gnu dot org
--- 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 -

[Bug fortran/34387] FAIL: gfortran.dg/optional_dim_2.f90: FE vs library argument missmatch

2008-01-03 Thread burnus at gcc dot gnu dot org
--- 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: