]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
re PR fortran/70524 (ICE when using -frepack-arrays -Warray-temporaries)
authorAndre Vehreschild <vehre@gcc.gnu.org>
Mon, 8 Aug 2016 08:11:29 +0000 (10:11 +0200)
committerAndre Vehreschild <vehre@gcc.gnu.org>
Mon, 8 Aug 2016 08:11:29 +0000 (10:11 +0200)
gcc/testsuite/ChangeLog:

2016-08-08  Andre Vehreschild  <vehre@gcc.gnu.org>

PR fortran/70524
* gfortran.dg/dependency_48.f90: New test.

gcc/fortran/ChangeLog:

2016-08-08  Andre Vehreschild  <vehre@gcc.gnu.org>

PR fortran/70524
* trans-array.c (gfc_trans_dummy_array_bias): Ensure that the
location information is correctly set.
* trans-decl.c (gfc_trans_deferred_vars): Set the locus of the
current construct early.

From-SVN: r239230

gcc/fortran/ChangeLog
gcc/fortran/trans-array.c
gcc/fortran/trans-decl.c
gcc/testsuite/ChangeLog
gcc/testsuite/gfortran.dg/dependency_48.f90 [new file with mode: 0644]

index 04726bb5dc1e4160ddba1cef1b1a7548ef0ae585..bf5ec12c49c49fcb546a8b3ab5ebf2b559c32f1f 100644 (file)
@@ -1,3 +1,11 @@
+2016-08-08  Andre Vehreschild  <vehre@gcc.gnu.org>
+
+       PR fortran/70524
+       * trans-array.c (gfc_trans_dummy_array_bias): Ensure that the
+       location information is correctly set.
+       * trans-decl.c (gfc_trans_deferred_vars): Set the locus of the
+       current construct early.
+
 2016-08-03  Fritz Reese  <fritzoreese@gmail.com>
 
        * lang.opt: New option -fdec-intrinsic-ints.
index 7572755a7a6ddcf1eb69ec5753dea9b50b043420..0a66f83562c95f229d6058028efcb341230d0d6e 100644 (file)
@@ -6103,7 +6103,12 @@ gfc_trans_dummy_array_bias (gfc_symbol * sym, tree tmpdesc,
       return;
     }
 
+  loc.nextc = NULL;
   gfc_save_backend_locus (&loc);
+  /* loc.nextc is not set by save_backend_locus but the location routines
+     depend on it.  */
+  if (loc.nextc == NULL)
+    loc.nextc = loc.lb->line;
   gfc_set_backend_locus (&sym->declared_at);
 
   /* Descriptor type.  */
index 05dfcb4be7044c5f372b9ad2f72a93c94b96f07e..2a34a4c23463eb694fbf224f0e100dd91ecebcc3 100644 (file)
@@ -4087,6 +4087,8 @@ gfc_trans_deferred_vars (gfc_symbol * proc_sym, gfc_wrapped_block * block)
       else if (proc_sym->as)
        {
          tree result = TREE_VALUE (current_fake_result_decl);
+         gfc_save_backend_locus (&loc);
+         gfc_set_backend_locus (&proc_sym->declared_at);
          gfc_trans_dummy_array_bias (proc_sym, result, block);
 
          /* An automatic character length, pointer array result.  */
@@ -4096,8 +4098,6 @@ gfc_trans_deferred_vars (gfc_symbol * proc_sym, gfc_wrapped_block * block)
              tmp = NULL;
              if (proc_sym->ts.deferred)
                {
-                 gfc_save_backend_locus (&loc);
-                 gfc_set_backend_locus (&proc_sym->declared_at);
                  gfc_start_block (&init);
                  tmp = gfc_null_and_pass_deferred_len (proc_sym, &init, &loc);
                  gfc_add_init_cleanup (block, gfc_finish_block (&init), tmp);
index 57a6c4ba34aaa31cb1c05a87bbbcd22b712fd9c2..5e5ab27a7fe30cdf3cb5a3a6b7a0e26894306b55 100644 (file)
@@ -1,3 +1,8 @@
+2016-08-08  Andre Vehreschild  <vehre@gcc.gnu.org>
+
+       PR fortran/70524
+       * gfortran.dg/dependency_48.f90: New test.
+
 2016-08-07  Thomas Koenig  <tkoenig@gcc.gnu.org>
 
        PR fortran/70040
diff --git a/gcc/testsuite/gfortran.dg/dependency_48.f90 b/gcc/testsuite/gfortran.dg/dependency_48.f90
new file mode 100644 (file)
index 0000000..6470019
--- /dev/null
@@ -0,0 +1,26 @@
+! { dg-do compile }
+! { dg-options "-frepack-arrays -Warray-temporaries -O" }
+
+! Same as dependency_35 but with repack-arrays
+
+module foo
+  implicit none
+contains
+  pure function bar(i,j) ! { dg-warning "Creating array temporary at \\(1\\)" }
+    integer, intent(in) :: i,j
+    integer, dimension(2,2) :: bar
+    bar = 33
+  end function bar
+end module foo
+
+program main
+  use foo
+  implicit none
+  integer a(2,2), b(2,2),c(2,2), d(2,2), e(2)
+
+  read (*,*) b, c, d
+  a = matmul(b,c) + d
+  a = b + bar(3,4)
+  a = bar(3,4)*5 + b
+  e = sum(b,1) + 3
+end program main