Message ID | 51BC86D1.5040005@sfr.fr |
---|---|
State | New |
Headers | show |
Mikael Morin wrote: > Dominique noticed that the patch also fixed PR56136 whose test is very > close to the one of PR49074. > This made me notice that while the PR56136 test should use a temporary > (and does), the PR49074 one shouldn't. That is fixed with the attached > patch. > Then the ICE which was fixed by the previous patch isn't > reachable any more in the PR49074 test-case, so I also add a test > based on the PR56136 one. > > Regression tested on x86_64-unknown-linux-gnu. OK for trunk? OK. Thanks to the patch - and for voiding the temporary. Tobias PS: Pending patch:Print exception status at STOP, http://gcc.gnu.org/ml/fortran/2013-06/msg00077.html
Index: dependency.c =================================================================== --- dependency.c (révision 200067) +++ dependency.c (copie de travail) @@ -990,7 +990,9 @@ gfc_check_argument_var_dependency (gfc_expr *var, return 0; case EXPR_ARRAY: - return gfc_check_dependency (var, expr, 1); + /* the scalarizer always generates a temporary for array constructors, + so there is no dependency. */ + return 0; case EXPR_FUNCTION: if (intent != INTENT_IN)