[Bug middle-end/17756] [4.0 Regression] contained_3.f90

[tobi at gcc dot gnu dot org]

------- Additional Comments From tobi at gcc dot gnu dot org  2004-10-04 19:25 -------
I can confirm that the gfortran testsuite looks much better with the preliminary
patch rth suggested on IRC:
Index: tree-ssa-operands.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/tree-ssa-operands.c,v
retrieving revision 2.48
diff -u -p -r2.48 tree-ssa-operands.c
--- tree-ssa-operands.c 26 Sep 2004 19:53:13 -0000      2.48
+++ tree-ssa-operands.c 4 Oct 2004 19:16:55 -0000
@@ -1004,6 +1004,7 @@ get_expr_operands (tree stmt, tree *expr
     case VAR_DECL:
     case PARM_DECL:
     case RESULT_DECL:
+    case CONST_DECL:
       /* If we found a variable, add it to DEFS or USES depending
         on the operand flags.  */
       add_stmt_operand (expr_p, stmt, flags);

I'll be adding new tests to the testcase
gfortran.fortran-torture/execute/intrinsic_pack.f90 shortly. At -O2 and higher
these tests fail with the same error David reported, and they pass with the
above patch. I'll let those tests commented out, so that no spurious regressions
appear in testsuite runs. Please uncomment them, once the patch for this bug is
in the tree.

The significant lines from the testcase are:
! this is waiting for PR 17756 to be fixed
!   call tests_with_temp()
Remove the first line, remove the exclamation mark from the second line.

Thanks.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tobi at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=17756