Support TYPE as alias for PRINT with -fdec.
authorFritz Reese <fritzoreese@gmail.com>
Tue, 25 Oct 2016 15:24:36 +0000 (15:24 +0000)
committerFritz Reese <foreese@gcc.gnu.org>
Tue, 25 Oct 2016 15:24:36 +0000 (15:24 +0000)
gcc/fortran/
* decl.c (gfc_match_type): New function.
* match.h (gfc_match_type): New function.
* match.c (gfc_match_if): Special case for one-line IFs.
* gfortran.texi: Update documentation.
* parse.c (decode_statement): Invoke gfc_match_type.

gcc/testsuite/gfortran.dg/
* dec_type_print.f90: New testcase.

From-SVN: r241518

gcc/fortran/ChangeLog
gcc/fortran/decl.c
gcc/fortran/gfortran.texi
gcc/fortran/match.c
gcc/fortran/match.h
gcc/fortran/parse.c
gcc/testsuite/ChangeLog
gcc/testsuite/gfortran.dg/dec_type_print.f90 [new file with mode: 0644]

index d3971a63610d9d7fd7667ccf6177845fb443a503..986eedfde2afcc4465649c7df9709a854b89f21c 100644 (file)
@@ -1,3 +1,11 @@
+2016-10-25  Fritz Reese <fritzoreese@gmail.com>
+
+       * decl.c (gfc_match_type): New function.
+       * match.h (gfc_match_type): New function.
+       * match.c (gfc_match_if): Special case for one-line IFs.
+       * gfortran.texi: Update documentation.
+       * parse.c (decode_statement): Invoke gfc_match_type.
+
 2016-10-25  Fritz Reese <fritzoreese@gmail.com>
 
        * gfortran.texi: Document.
index e47d8ede33cf5368f7d5abd99e93f58c18d51105..6c9d0570df73651993ee7b18ad1e62d646457f4e 100644 (file)
@@ -8710,6 +8710,100 @@ gfc_match_structure_decl (void)
     return MATCH_YES;
 }
 
+
+/* This function does some work to determine which matcher should be used to
+ * match a statement beginning with "TYPE". This is used to disambiguate TYPE
+ * as an alias for PRINT from derived type declarations, TYPE IS statements,
+ * and derived type data declarations.  */
+
+match
+gfc_match_type (gfc_statement *st)
+{
+  char name[GFC_MAX_SYMBOL_LEN + 1];
+  match m;
+  locus old_loc;
+
+  /* Requires -fdec.  */
+  if (!flag_dec)
+    return MATCH_NO;
+
+  m = gfc_match ("type");
+  if (m != MATCH_YES)
+    return m;
+  /* If we already have an error in the buffer, it is probably from failing to
+   * match a derived type data declaration. Let it happen.  */
+  else if (gfc_error_flag_test ())
+    return MATCH_NO;
+
+  old_loc = gfc_current_locus;
+  *st = ST_NONE;
+
+  /* If we see an attribute list before anything else it's definitely a derived
+   * type declaration.  */
+  if (gfc_match (" ,") == MATCH_YES || gfc_match (" ::") == MATCH_YES)
+    {
+      gfc_current_locus = old_loc;
+      *st = ST_DERIVED_DECL;
+      return gfc_match_derived_decl ();
+    }
+
+  /* By now "TYPE" has already been matched. If we do not see a name, this may
+   * be something like "TYPE *" or "TYPE <fmt>".  */
+  m = gfc_match_name (name);
+  if (m != MATCH_YES)
+    {
+      /* Let print match if it can, otherwise throw an error from
+       * gfc_match_derived_decl.  */
+      gfc_current_locus = old_loc;
+      if (gfc_match_print () == MATCH_YES)
+       {
+         *st = ST_WRITE;
+         return MATCH_YES;
+       }
+      gfc_current_locus = old_loc;
+      *st = ST_DERIVED_DECL;
+      return gfc_match_derived_decl ();
+    }
+
+  /* A derived type declaration requires an EOS. Without it, assume print.  */
+  m = gfc_match_eos ();
+  if (m == MATCH_NO)
+    {
+      /* Check manually for TYPE IS (... - this is invalid print syntax.  */
+      if (strncmp ("is", name, 3) == 0
+         && gfc_match (" (", name) == MATCH_YES)
+       {
+         gfc_current_locus = old_loc;
+         gcc_assert (gfc_match (" is") == MATCH_YES);
+         *st = ST_TYPE_IS;
+         return gfc_match_type_is ();
+       }
+      gfc_current_locus = old_loc;
+      *st = ST_WRITE;
+      return gfc_match_print ();
+    }
+  else
+    {
+      /* By now we have "TYPE <name> <EOS>". Check first if the name is an
+       * intrinsic typename - if so let gfc_match_derived_decl dump an error.
+       * Otherwise if gfc_match_derived_decl fails it's probably an existing
+       * symbol which can be printed.  */
+      gfc_current_locus = old_loc;
+      m = gfc_match_derived_decl ();
+      if (gfc_is_intrinsic_typename (name) || m == MATCH_YES)
+       {
+         *st = ST_DERIVED_DECL;
+         return m;
+       }
+      gfc_current_locus = old_loc;
+      *st = ST_WRITE;
+      return gfc_match_print ();
+    }
+
+  return MATCH_NO;
+}
+
+
 /* Match the beginning of a derived type declaration.  If a type name
    was the result of a function, then it is possible to have a symbol
    already to be known as a derived type yet have no components.  */
index 688b9565e26ef635f1e98733f07a277b8705406c..fb47c13ceaaf311b85eab0188b3c9f5635540c2f 100644 (file)
@@ -1466,6 +1466,7 @@ compatibility extensions along with those enabled by @option{-std=legacy}.
 * AUTOMATIC and STATIC attributes::
 * Extended math intrinsics::
 * Form feed as whitespace::
+* TYPE as an alias for PRINT::
 @end menu
 
 @node Old-style kind specifications
@@ -2521,6 +2522,21 @@ though the Fortran standard does not mention this. GNU Fortran supports the
 interpretation of form feed characters in source as whitespace for
 compatibility.
 
+@node TYPE as an alias for PRINT
+@subsection TYPE as an alias for PRINT
+@cindex type alias print
+For compatibility, GNU Fortran will interpret @code{TYPE} statements as
+@code{PRINT} statements with the flag @option{-fdec}.  With this flag asserted,
+the following two examples are equivalent:
+
+@smallexample
+TYPE *, 'hello world'
+@end smallexample
+
+@smallexample
+PRINT *, 'hello world'
+@end smallexample
+
 
 @node Extensions not implemented in GNU Fortran
 @section Extensions not implemented in GNU Fortran
index aa9961c6ed5ac3fd0a6a369073ab9a55c9ec1b66..236231e3ee69795db8677cc07c7989ae84544416 100644 (file)
@@ -1622,6 +1622,9 @@ gfc_match_if (gfc_statement *if_type)
   match ("where", match_simple_where, ST_WHERE)
   match ("write", gfc_match_write, ST_WRITE)
 
+  if (flag_dec)
+    match ("type", gfc_match_print, ST_WRITE)
+
   /* The gfc_match_assignment() above may have returned a MATCH_NO
      where the assignment was to a named constant.  Check that
      special case here.  */
index 24131635713d9e580c7af7e9e4c781ffedb9aea6..eeb26931567c6b80159ad395e2b444ab7fe7e977 100644 (file)
@@ -214,6 +214,7 @@ match gfc_match_union (void);
 match gfc_match_structure_decl (void);
 match gfc_match_derived_decl (void);
 match gfc_match_final_decl (void);
+match gfc_match_type (gfc_statement *);
 
 match gfc_match_implicit_none (void);
 match gfc_match_implicit (void);
index 03234358547b8e56990c2d32fbcc6560c58549fc..760d3afdb5f02aa7d299137482751f1f8facf696 100644 (file)
@@ -413,6 +413,12 @@ decode_statement (void)
   gfc_undo_symbols ();
   gfc_current_locus = old_locus;
 
+  /* Try to match TYPE as an alias for PRINT.  */
+  if (gfc_match_type (&st) == MATCH_YES)
+    return st;
+  gfc_undo_symbols ();
+  gfc_current_locus = old_locus;
+
   match (NULL, gfc_match_do, ST_DO);
   match (NULL, gfc_match_block, ST_BLOCK);
   match (NULL, gfc_match_associate, ST_ASSOCIATE);
index b2662db600803c176e66251ef1fc2575cf502c63..a64e74d91d9670e91a708a01351c7d3ed1cbaf71 100644 (file)
@@ -1,7 +1,11 @@
 2016-10-25  Fritz Reese <fritzoreese@gmail.com>
 
-       gfortran.dg/
-       * feed_1.f90, feed_2.f90: New testcases.
+       * gfortran.dg/dec_type_print.f90: New testcase.
+
+2016-10-25  Fritz Reese <fritzoreese@gmail.com>
+
+       * gfortran.dg/feed_1.f90: New test.
+       * gfortran.dg/feed_2.f90: New test.
 
 2016-10-25  Martin Liska  <mliska@suse.cz>
 
diff --git a/gcc/testsuite/gfortran.dg/dec_type_print.f90 b/gcc/testsuite/gfortran.dg/dec_type_print.f90
new file mode 100644 (file)
index 0000000..ca40798
--- /dev/null
@@ -0,0 +1,84 @@
+! { dg-do compile }
+! { dg-options "-fdec" }
+!
+! Test the usage of TYPE as an alias for PRINT.
+!
+! Note the heavy use of other TYPE statements to test for
+! regressions involving ambiguity.
+!
+program main
+
+logical bool
+integer i /0/, j /1/, k /2/
+character(*), parameter :: fmtstr = "(A11)"
+namelist /nmlist/ i, j, k
+integer, parameter :: n = 5
+real a(n)
+
+! derived type declarations
+type is
+  integer i
+end type
+
+type point
+   real x, y
+end type point
+
+type, extends(point) :: point_3d
+   real :: z
+end type point_3d
+
+type, extends(point) :: color_point
+   integer :: color
+end type color_point
+
+! declaration type specification
+type(is) x
+type(point), target :: p
+type(point_3d), target :: p3
+type(color_point), target :: c
+class(point), pointer :: p_or_c
+
+! select type
+p_or_c => c
+select type ( a => p_or_c )
+  class is ( point )
+    print *, "point"     ! <===
+  type is ( point_3d )
+    print *, "point 3D"
+end select
+
+! Type as alias for print
+type*
+type *
+type*,'St','ar'
+type *, 'St', 'ar'
+type 10, 'Integer literal'
+type 10, 'Integer variable'
+type '(A11)', 'Character literal'
+type fmtstr, 'Character variable'
+type nmlist ! namelist
+
+a(1) = 0
+call f(.true., a, n)
+
+10    format (A11)
+
+end program
+
+
+subroutine f(b,a,n)
+  implicit none
+  logical b
+  real a(*)
+  integer n
+
+  integer i
+
+  do i = 2,n
+    a(i) = 2 * (a(i-1) + 1)
+    if (b) type*,a(i) ! test TYPE as PRINT inside one-line IF
+  enddo
+
+  return
+end subroutine