Re: [patch, fortran] Handle missing optional MASK for intrinsics

2018-12-31 Thread Dominique d'Humières
Hi Thomas, Another glitch: the following test (reduced from gfortran.dg/optional_absent_4.f90) module z implicit none contains subroutine findloc_4 (input, val, res, mask) logical, intent(in), optional :: mask(:,:) integer, intent(in) :: input(:,:) integer, dimension(:), intent(

Re: [patch, fortran] Handle missing optional MASK for intrinsics

2018-12-31 Thread Dominique d'Humières
Hi Thomas, There are some duplicate STOPs (6 and 10) in the test gfortran.dg/optional_absent_4.f90. Thanks for the patch, Dominique

Re: [patch, fortran] Handle missing optional MASK for intrinsics

2018-12-30 Thread Steve Kargl
On Sun, Dec 30, 2018 at 09:37:20AM -0800, Steve Kargl wrote: > On Sun, Dec 30, 2018 at 06:10:14PM +0100, Thomas Koenig wrote: > > > + { > > + tree present; > > + tree type; > > + > > + type = TREE_TYPE (maskse.expr); > > + present = gfc_conv_expr_present (maskexpr->symtree->n.sym

Re: [patch, fortran] Handle missing optional MASK for intrinsics

2018-12-30 Thread Steve Kargl
On Sun, Dec 30, 2018 at 06:10:14PM +0100, Thomas Koenig wrote: > Index: gcc/fortran/trans-expr.c > === > --- gcc/fortran/trans-expr.c (Revision 267347) > +++ gcc/fortran/trans-expr.c (Arbeitskopie) > @@ -5760,17 +5760,21 @@ gfc_conv