/* Deal with I/O statements & related stuff.
- Copyright (C) 2000-2018 Free Software Foundation, Inc.
+ Copyright (C) 2000-2021 Free Software Foundation, Inc.
Contributed by Andy Vaught
This file is part of GCC.
#include "gfortran.h"
#include "match.h"
#include "parse.h"
+#include "constructor.h"
gfc_st_label
format_asterisk = {0, NULL, NULL, -1, ST_LABEL_FORMAT, ST_LABEL_FORMAT, NULL,
#define RESOLVE_TAG(x, y) if (!resolve_tag (x, y)) return false;
-/* Are we currently processing an asynchronous I/O statement? */
-
-bool async_io_dt;
-
/**************** Fortran 95 FORMAT parser *****************/
/* FORMAT tokens returned by format_lex(). */
static bool
check_format (bool is_input)
{
- const char *posint_required = _("Positive width required");
- const char *nonneg_required = _("Nonnegative width required");
- const char *unexpected_element = _("Unexpected element %qc in format "
- "string at %L");
- const char *unexpected_end = _("Unexpected end of format string");
- const char *zero_width = _("Zero width in format descriptor");
+ const char *posint_required
+ = G_("Positive width required in format string at %L");
+ const char *nonneg_required
+ = G_("Nonnegative width required in format string at %L");
+ const char *unexpected_element
+ = G_("Unexpected element %qc in format string at %L");
+ const char *unexpected_end
+ = G_("Unexpected end of format string in format string at %L");
+ const char *zero_width
+ = G_("Zero width in format descriptor in format string at %L");
const char *error = NULL;
format_token t, u;
goto fail;
if (t != FMT_LPAREN)
{
- error = _("Missing leading left parenthesis");
+ error = G_("Missing leading left parenthesis in format string at %L");
goto syntax;
}
level++;
goto format_item;
}
- error = _("Left parenthesis required after %<*%>");
+ error = G_("Left parenthesis required after %<*%> in format string "
+ "at %L");
goto syntax;
case FMT_POSINT:
goto fail;
if (t != FMT_P)
{
- error = _("Expected P edit descriptor");
+ error = G_("Expected P edit descriptor in format string at %L");
goto syntax;
}
case FMT_P:
/* P requires a prior number. */
- error = _("P descriptor requires leading scale factor");
+ error = G_("P descriptor requires leading scale factor in format "
+ "string at %L");
goto syntax;
case FMT_X:
error = unexpected_end;
goto syntax;
+ case FMT_RPAREN:
+ if (flag_dec_blank_format_item)
+ goto finished;
+ else
+ {
+ error = G_("Missing item in format string at %L");
+ goto syntax;
+ }
+
default:
error = unexpected_element;
goto syntax;
&& t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES
&& t != FMT_D && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
{
- error = _("Comma required after P descriptor");
+ error = G_("Comma required after P descriptor in format string "
+ "at %L");
goto syntax;
}
if (t != FMT_COMMA)
if (t == FMT_ERROR)
goto fail;
}
- if (t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES && t != FMT_D
- && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
+ if (t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES
+ && t != FMT_D && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
{
- error = _("Comma required after P descriptor");
+ error = G_("Comma required after P descriptor in format string "
+ "at %L");
goto syntax;
}
}
t = format_lex ();
if (t != FMT_POSINT)
{
- error = _("Positive width required with T descriptor");
+ error = G_("Positive width required with T descriptor in format "
+ "string at %L");
goto syntax;
}
break;
u = format_lex ();
if (u == FMT_E)
{
- error = _("E specifier not allowed with g0 descriptor");
+ error = G_("E specifier not allowed with g0 descriptor in "
+ "format string at %L");
goto syntax;
}
saved_token = u;
if (u != FMT_POSINT)
{
+ if (flag_dec)
+ {
+ if (flag_dec_format_defaults)
+ {
+ /* Assume a default width based on the variable size. */
+ saved_token = u;
+ break;
+ }
+ else
+ {
+ gfc_error ("Positive width required in format "
+ "specifier %s at %L", token_to_string (t),
+ &format_locus);
+ saved_token = u;
+ goto fail;
+ }
+ }
+
format_locus.nextc += format_string_pos;
- gfc_error ("Positive width required in format "
- "specifier %s at %L", token_to_string (t),
- &format_locus);
- saved_token = u;
- goto fail;
+ if (!gfc_notify_std (GFC_STD_F2018,
+ "positive width required at %L",
+ &format_locus))
+ {
+ saved_token = u;
+ goto fail;
+ }
+ if (flag_dec_format_defaults)
+ {
+ /* Assume a default width based on the variable size. */
+ saved_token = u;
+ break;
+ }
}
u = format_lex ();
if (u == FMT_ERROR)
goto fail;
if (u != FMT_E)
- {
- saved_token = u;
- }
+ saved_token = u;
else
{
u = format_lex ();
goto fail;
if (u != FMT_POSINT)
{
- error = _("Positive exponent width required");
- goto syntax;
+ if (u == FMT_ZERO)
+ {
+ if (!gfc_notify_std (GFC_STD_F2018,
+ "Positive exponent width required in "
+ "format string at %L", &format_locus))
+ {
+ saved_token = u;
+ goto fail;
+ }
+ }
+ else
+ {
+ error = G_("Positive exponent width required in format "
+ "string at %L");
+ goto syntax;
+ }
}
}
goto dtio_vlist;
if (t != FMT_RPAREN)
{
- error = _("Right parenthesis expected at %C");
+ error = G_("Right parenthesis expected at %C in format string "
+ "at %L");
goto syntax;
}
goto between_desc;
goto fail;
if (t != FMT_ZERO && t != FMT_POSINT)
{
+ if (flag_dec_format_defaults)
+ {
+ /* Assume the default width is expected here and continue lexing. */
+ value = 0; /* It doesn't matter what we set the value to here. */
+ saved_token = t;
+ break;
+ }
error = nonneg_required;
goto syntax;
}
/* Warn if -std=legacy, otherwise error. */
if (gfc_option.warn_std != 0)
{
- error = _("Period required in format specifier");
+ error = G_("Period required in format specifier in format "
+ "string at %L");
goto syntax;
}
if (mode != MODE_FORMAT)
goto fail;
if (t != FMT_ZERO && t != FMT_POSINT)
{
- error = nonneg_required;
- goto syntax;
+ if (flag_dec_format_defaults)
+ {
+ /* Assume the default width is expected here and continue lexing. */
+ value = 0; /* It doesn't matter what we set the value to here. */
+ saved_token = t;
+ }
+ else
+ {
+ error = nonneg_required;
+ goto syntax;
+ }
}
else if (is_input && t == FMT_ZERO)
{
if (t == FMT_ERROR)
goto fail;
if (t != FMT_PERIOD)
- {
- saved_token = t;
- }
+ saved_token = t;
else
{
t = format_lex ();
if (error == unexpected_element)
gfc_error (error, error_element, &format_locus);
else
- gfc_error ("%s in format string at %L", error, &format_locus);
+ gfc_error (error, &format_locus);
fail:
rv = false;
}
-/* Check for a CHARACTER variable. The check for scalar is done in
- resolve_tag. */
-
-static bool
-check_char_variable (gfc_expr *e)
-{
- if (e->expr_type != EXPR_VARIABLE || e->ts.type != BT_CHARACTER)
- {
- gfc_error("IOMSG must be a scalar-default-char-variable at %L", &e->where);
- return false;
- }
- return true;
-}
-
-
-static bool
-is_char_type (const char *name, gfc_expr *e)
-{
- gfc_resolve_expr (e);
-
- if (e->ts.type != BT_CHARACTER)
- {
- gfc_error ("%s requires a scalar-default-char-expr at %L",
- name, &e->where);
- return false;
- }
- return true;
-}
-
-
/* Match an expression I/O tag of some sort. */
static match
return MATCH_ERROR;
}
- if (result->symtree->n.sym->attr.intent == INTENT_IN)
+ if (result->symtree)
{
- gfc_error ("Variable %s cannot be INTENT(IN) at %C", tag->name);
- gfc_free_expr (result);
- return MATCH_ERROR;
- }
+ bool impure;
- bool impure = gfc_impure_variable (result->symtree->n.sym);
- if (impure && gfc_pure (NULL))
- {
- gfc_error ("Variable %s cannot be assigned in PURE procedure at %C",
- tag->name);
- gfc_free_expr (result);
- return MATCH_ERROR;
- }
+ if (result->symtree->n.sym->attr.intent == INTENT_IN)
+ {
+ gfc_error ("Variable %s cannot be INTENT(IN) at %C", tag->name);
+ gfc_free_expr (result);
+ return MATCH_ERROR;
+ }
+
+ impure = gfc_impure_variable (result->symtree->n.sym);
+ if (impure && gfc_pure (NULL))
+ {
+ gfc_error ("Variable %s cannot be assigned in PURE procedure at %C",
+ tag->name);
+ gfc_free_expr (result);
+ return MATCH_ERROR;
+ }
- if (impure)
- gfc_unset_implicit_pure (NULL);
+ if (impure)
+ gfc_unset_implicit_pure (NULL);
+ }
*v = result;
return MATCH_YES;
m = match_vtag (tag, result);
if (m == MATCH_YES)
- gfc_check_do_variable ((*result)->symtree);
+ {
+ if ((*result)->symtree)
+ gfc_check_do_variable ((*result)->symtree);
+
+ if ((*result)->expr_type == EXPR_CONSTANT)
+ {
+ gfc_error ("Expecting a variable at %L", &(*result)->where);
+ return MATCH_ERROR;
+ }
+ }
return m;
}
/* Resolution of the FORMAT tag, to be called from resolve_tag. */
static bool
-resolve_tag_format (const gfc_expr *e)
+resolve_tag_format (gfc_expr *e)
{
if (e->expr_type == EXPR_CONSTANT
&& (e->ts.type != BT_CHARACTER
return false;
}
+ /* Concatenate a constant character array into a single character
+ expression. */
+
+ if ((e->expr_type == EXPR_ARRAY || e->rank > 0)
+ && e->ts.type == BT_CHARACTER
+ && gfc_is_constant_expr (e))
+ {
+ if (e->expr_type == EXPR_VARIABLE
+ && e->symtree->n.sym->attr.flavor == FL_PARAMETER)
+ gfc_simplify_expr (e, 1);
+
+ if (e->expr_type == EXPR_ARRAY)
+ {
+ gfc_constructor *c;
+ gfc_charlen_t n, len;
+ gfc_expr *r;
+ gfc_char_t *dest, *src;
+
+ if (e->value.constructor == NULL)
+ {
+ gfc_error ("FORMAT tag at %L cannot be a zero-sized array",
+ &e->where);
+ return false;
+ }
+
+ n = 0;
+ c = gfc_constructor_first (e->value.constructor);
+ len = c->expr->value.character.length;
+
+ for ( ; c; c = gfc_constructor_next (c))
+ n += len;
+
+ r = gfc_get_character_expr (e->ts.kind, &e->where, NULL, n);
+ dest = r->value.character.string;
+
+ for (c = gfc_constructor_first (e->value.constructor);
+ c; c = gfc_constructor_next (c))
+ {
+ src = c->expr->value.character.string;
+ for (gfc_charlen_t i = 0 ; i < len; i++)
+ *dest++ = *src++;
+ }
+
+ gfc_replace_expr (e, r);
+ return true;
+ }
+ }
+
/* If e's rank is zero and e is not an element of an array, it should be
of integer or character type. The integer variable should be
ASSIGNED. */
match m;
m = match_etag (&tag_e_async, &open->asynchronous);
- if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", open->asynchronous))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
m = match_etag (&tag_unit, &open->unit);
if (m != MATCH_NO)
return m;
m = match_etag (&tag_iomsg, &open->iomsg);
- if (m == MATCH_YES && !check_char_variable (open->iomsg))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
m = match_out_tag (&tag_iostat, &open->iostat);
}
+static int
+compare_to_allowed_values (const char *specifier, const char *allowed[],
+ const char *allowed_f2003[],
+ const char *allowed_gnu[], gfc_char_t *value,
+ const char *statement, bool warn, locus *where,
+ int *num = NULL);
+
+
+static bool
+check_open_constraints (gfc_open *open, locus *where);
+
/* Resolve everything in a gfc_open structure. */
bool
-gfc_resolve_open (gfc_open *open)
+gfc_resolve_open (gfc_open *open, locus *where)
{
-
RESOLVE_TAG (&tag_unit, open->unit);
RESOLVE_TAG (&tag_iomsg, open->iomsg);
RESOLVE_TAG (&tag_iostat, open->iostat);
if (!gfc_reference_st_label (open->err, ST_LABEL_TARGET))
return false;
- return true;
+ return check_open_constraints (open, where);
}
allowed in F95 or F2003, issuing an error message and returning a zero
value if it is not allowed. */
-static int
-compare_to_allowed_values (const char *specifier, const char *allowed[],
- const char *allowed_f2003[],
- const char *allowed_gnu[], gfc_char_t *value,
- const char *statement, bool warn,
- int *num = NULL);
-
static int
compare_to_allowed_values (const char *specifier, const char *allowed[],
- const char *allowed_f2003[],
+ const char *allowed_f2003[],
const char *allowed_gnu[], gfc_char_t *value,
- const char *statement, bool warn, int *num)
+ const char *statement, bool warn, locus *where,
+ int *num)
{
int i;
unsigned int len;
return 1;
}
+ if (!where)
+ where = &gfc_current_locus;
+
for (i = 0; allowed_f2003 && allowed_f2003[i]; i++)
if (len == strlen (allowed_f2003[i])
&& gfc_wide_strncasecmp (value, allowed_f2003[i],
if (n == WARNING || (warn && n == ERROR))
{
- gfc_warning (0, "Fortran 2003: %s specifier in %s statement at %C "
- "has value %qs", specifier, statement,
+ gfc_warning (0, "Fortran 2003: %s specifier in %s statement at %L "
+ "has value %qs", specifier, statement, where,
allowed_f2003[i]);
return 1;
}
if (n == ERROR)
{
gfc_notify_std (GFC_STD_F2003, "%s specifier in "
- "%s statement at %C has value %qs", specifier,
- statement, allowed_f2003[i]);
+ "%s statement at %L has value %qs", specifier,
+ statement, where, allowed_f2003[i]);
return 0;
}
if (n == WARNING || (warn && n == ERROR))
{
- gfc_warning (0, "Extension: %s specifier in %s statement at %C "
- "has value %qs", specifier, statement,
+ gfc_warning (0, "Extension: %s specifier in %s statement at %L "
+ "has value %qs", specifier, statement, where,
allowed_gnu[i]);
return 1;
}
if (n == ERROR)
{
gfc_notify_std (GFC_STD_GNU, "%s specifier in "
- "%s statement at %C has value %qs", specifier,
- statement, allowed_gnu[i]);
+ "%s statement at %L has value %qs", specifier,
+ statement, where, allowed_gnu[i]);
return 0;
}
{
char *s = gfc_widechar_to_char (value, -1);
gfc_warning (0,
- "%s specifier in %s statement at %C has invalid value %qs",
- specifier, statement, s);
+ "%s specifier in %s statement at %L has invalid value %qs",
+ specifier, statement, where, s);
free (s);
return 1;
}
else
{
char *s = gfc_widechar_to_char (value, -1);
- gfc_error ("%s specifier in %s statement at %C has invalid value %qs",
- specifier, statement, s);
+ gfc_error ("%s specifier in %s statement at %L has invalid value %qs",
+ specifier, statement, where, s);
free (s);
return 0;
}
}
-/* Match an OPEN statement. */
+/* Check constraints on the OPEN statement.
+ Similar to check_io_constraints for data transfer statements.
+ At this point all tags have already been resolved via resolve_tag, which,
+ among other things, verifies that BT_CHARACTER tags are of default kind. */
-match
-gfc_match_open (void)
+static bool
+check_open_constraints (gfc_open *open, locus *where)
{
- gfc_open *open;
- match m;
- bool warn;
-
- m = gfc_match_char ('(');
- if (m == MATCH_NO)
- return m;
-
- open = XCNEW (gfc_open);
-
- m = match_open_element (open);
-
- if (m == MATCH_ERROR)
- goto cleanup;
- if (m == MATCH_NO)
- {
- m = gfc_match_expr (&open->unit);
- if (m == MATCH_ERROR)
- goto cleanup;
- }
-
- for (;;)
- {
- if (gfc_match_char (')') == MATCH_YES)
- break;
- if (gfc_match_char (',') != MATCH_YES)
- goto syntax;
-
- m = match_open_element (open);
- if (m == MATCH_ERROR)
- goto cleanup;
- if (m == MATCH_NO)
- goto syntax;
- }
-
- if (gfc_match_eos () == MATCH_NO)
- goto syntax;
-
- if (gfc_pure (NULL))
- {
- gfc_error ("OPEN statement not allowed in PURE procedure at %C");
- goto cleanup;
- }
-
- gfc_unset_implicit_pure (NULL);
-
- warn = (open->err || open->iostat) ? true : false;
-
- /* Checks on NEWUNIT specifier. */
- if (open->newunit)
- {
- if (open->unit)
- {
- gfc_error ("UNIT specifier not allowed with NEWUNIT at %C");
- goto cleanup;
- }
+#define warn_or_error(...) \
+{ \
+ if (warn) \
+ gfc_warning (0, __VA_ARGS__); \
+ else \
+ { \
+ gfc_error (__VA_ARGS__); \
+ return false; \
+ } \
+}
- if (!open->file && open->status)
- {
- if (open->status->expr_type == EXPR_CONSTANT
- && gfc_wide_strncasecmp (open->status->value.character.string,
- "scratch", 7) != 0)
- {
- gfc_error ("NEWUNIT specifier must have FILE= "
- "or STATUS='scratch' at %C");
- goto cleanup;
- }
- }
- }
- else if (!open->unit)
- {
- gfc_error ("OPEN statement at %C must have UNIT or NEWUNIT specified");
- goto cleanup;
- }
+ bool warn = (open->err || open->iostat) ? true : false;
/* Checks on the ACCESS specifier. */
if (open->access && open->access->expr_type == EXPR_CONSTANT)
static const char *access_f2003[] = { "STREAM", NULL };
static const char *access_gnu[] = { "APPEND", NULL };
- if (!is_char_type ("ACCESS", open->access))
- goto cleanup;
-
if (!compare_to_allowed_values ("ACCESS", access_f95, access_f2003,
access_gnu,
open->access->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->access->where))
+ return false;
}
/* Checks on the ACTION specifier. */
gfc_char_t *str = open->action->value.character.string;
static const char *action[] = { "READ", "WRITE", "READWRITE", NULL };
- if (!is_char_type ("ACTION", open->action))
- goto cleanup;
-
if (!compare_to_allowed_values ("ACTION", action, NULL, NULL,
- str, "OPEN", warn))
- goto cleanup;
+ str, "OPEN", warn, &open->action->where))
+ return false;
/* With READONLY, only allow ACTION='READ'. */
if (open->readonly && (gfc_wide_strlen (str) != 4
|| gfc_wide_strncasecmp (str, "READ", 4) != 0))
{
- gfc_error ("ACTION type conflicts with READONLY specifier at %C");
- goto cleanup;
+ gfc_error ("ACTION type conflicts with READONLY specifier at %L",
+ &open->action->where);
+ return false;
}
}
+
/* If we see READONLY and no ACTION, set ACTION='READ'. */
else if (open->readonly && open->action == NULL)
{
/* Checks on the ASYNCHRONOUS specifier. */
if (open->asynchronous)
{
- if (!gfc_notify_std (GFC_STD_F2003, "ASYNCHRONOUS= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("ASYNCHRONOUS", open->asynchronous))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "ASYNCHRONOUS= at %L "
+ "not allowed in Fortran 95",
+ &open->asynchronous->where))
+ return false;
if (open->asynchronous->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("ASYNCHRONOUS", asynchronous,
NULL, NULL, open->asynchronous->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->asynchronous->where))
+ return false;
}
}
/* Checks on the BLANK specifier. */
if (open->blank)
{
- if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("BLANK", open->blank))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %L "
+ "not allowed in Fortran 95", &open->blank->where))
+ return false;
if (open->blank->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("BLANK", blank, NULL, NULL,
open->blank->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->blank->where))
+ return false;
}
}
/* Checks on the CARRIAGECONTROL specifier. */
- if (open->cc)
+ if (open->cc && open->cc->expr_type == EXPR_CONSTANT)
{
- if (!is_char_type ("CARRIAGECONTROL", open->cc))
- goto cleanup;
-
- if (open->cc->expr_type == EXPR_CONSTANT)
- {
- static const char *cc[] = { "LIST", "FORTRAN", "NONE", NULL };
- if (!compare_to_allowed_values ("CARRIAGECONTROL", cc, NULL, NULL,
- open->cc->value.character.string,
- "OPEN", warn))
- goto cleanup;
- }
+ static const char *cc[] = { "LIST", "FORTRAN", "NONE", NULL };
+ if (!compare_to_allowed_values ("CARRIAGECONTROL", cc, NULL, NULL,
+ open->cc->value.character.string,
+ "OPEN", warn, &open->cc->where))
+ return false;
}
/* Checks on the DECIMAL specifier. */
if (open->decimal)
{
- if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("DECIMAL", open->decimal))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %L "
+ "not allowed in Fortran 95", &open->decimal->where))
+ return false;
if (open->decimal->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("DECIMAL", decimal, NULL, NULL,
open->decimal->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->decimal->where))
+ return false;
}
}
{
static const char *delim[] = { "APOSTROPHE", "QUOTE", "NONE", NULL };
- if (!is_char_type ("DELIM", open->delim))
- goto cleanup;
-
if (!compare_to_allowed_values ("DELIM", delim, NULL, NULL,
open->delim->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->delim->where))
+ return false;
}
}
/* Checks on the ENCODING specifier. */
if (open->encoding)
{
- if (!gfc_notify_std (GFC_STD_F2003, "ENCODING= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("ENCODING", open->encoding))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "ENCODING= at %L "
+ "not allowed in Fortran 95", &open->encoding->where))
+ return false;
if (open->encoding->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("ENCODING", encoding, NULL, NULL,
open->encoding->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->encoding->where))
+ return false;
}
}
{
static const char *form[] = { "FORMATTED", "UNFORMATTED", NULL };
- if (!is_char_type ("FORM", open->form))
- goto cleanup;
-
if (!compare_to_allowed_values ("FORM", form, NULL, NULL,
open->form->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->form->where))
+ return false;
}
/* Checks on the PAD specifier. */
{
static const char *pad[] = { "YES", "NO", NULL };
- if (!is_char_type ("PAD", open->pad))
- goto cleanup;
-
if (!compare_to_allowed_values ("PAD", pad, NULL, NULL,
open->pad->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->pad->where))
+ return false;
}
/* Checks on the POSITION specifier. */
{
static const char *position[] = { "ASIS", "REWIND", "APPEND", NULL };
- if (!is_char_type ("POSITION", open->position))
- goto cleanup;
-
if (!compare_to_allowed_values ("POSITION", position, NULL, NULL,
open->position->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->position->where))
+ return false;
}
/* Checks on the ROUND specifier. */
if (open->round)
{
- if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("ROUND", open->round))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %L "
+ "not allowed in Fortran 95", &open->round->where))
+ return false;
if (open->round->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("ROUND", round, NULL, NULL,
open->round->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->round->where))
+ return false;
}
}
/* Checks on the SHARE specifier. */
- if (open->share)
+ if (open->share && open->share->expr_type == EXPR_CONSTANT)
{
- if (!is_char_type ("SHARE", open->share))
- goto cleanup;
-
- if (open->share->expr_type == EXPR_CONSTANT)
- {
- static const char *share[] = { "DENYNONE", "DENYRW", NULL };
- if (!compare_to_allowed_values ("SHARE", share, NULL, NULL,
- open->share->value.character.string,
- "OPEN", warn))
- goto cleanup;
- }
+ static const char *share[] = { "DENYNONE", "DENYRW", NULL };
+ if (!compare_to_allowed_values ("SHARE", share, NULL, NULL,
+ open->share->value.character.string,
+ "OPEN", warn, &open->share->where))
+ return false;
}
/* Checks on the SIGN specifier. */
- if (open->sign)
+ if (open->sign)
{
- if (!gfc_notify_std (GFC_STD_F2003, "SIGN= at %C "
- "not allowed in Fortran 95"))
- goto cleanup;
-
- if (!is_char_type ("SIGN", open->sign))
- goto cleanup;
+ if (!gfc_notify_std (GFC_STD_F2003, "SIGN= at %L "
+ "not allowed in Fortran 95", &open->sign->where))
+ return false;
if (open->sign->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("SIGN", sign, NULL, NULL,
open->sign->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->sign->where))
+ return false;
}
}
-#define warn_or_error(...) \
-{ \
- if (warn) \
- gfc_warning (0, __VA_ARGS__); \
- else \
- { \
- gfc_error (__VA_ARGS__); \
- goto cleanup; \
- } \
-}
-
/* Checks on the RECL specifier. */
if (open->recl && open->recl->expr_type == EXPR_CONSTANT
&& open->recl->ts.type == BT_INTEGER
&& mpz_sgn (open->recl->value.integer) != 1)
{
- warn_or_error ("RECL in OPEN statement at %C must be positive");
+ warn_or_error (G_("RECL in OPEN statement at %L must be positive"),
+ &open->recl->where);
}
/* Checks on the STATUS specifier. */
static const char *status[] = { "OLD", "NEW", "SCRATCH",
"REPLACE", "UNKNOWN", NULL };
- if (!is_char_type ("STATUS", open->status))
- goto cleanup;
-
if (!compare_to_allowed_values ("STATUS", status, NULL, NULL,
open->status->value.character.string,
- "OPEN", warn))
- goto cleanup;
+ "OPEN", warn, &open->status->where))
+ return false;
/* F2003, 9.4.5: If the STATUS= specifier has the value NEW or REPLACE,
the FILE= specifier shall appear. */
{
char *s = gfc_widechar_to_char (open->status->value.character.string,
-1);
- warn_or_error ("The STATUS specified in OPEN statement at %C is "
- "%qs and no FILE specifier is present", s);
+ warn_or_error (G_("The STATUS specified in OPEN statement at %L is "
+ "%qs and no FILE specifier is present"),
+ &open->status->where, s);
free (s);
}
if (gfc_wide_strncasecmp (open->status->value.character.string,
"scratch", 7) == 0 && open->file)
{
- warn_or_error ("The STATUS specified in OPEN statement at %C "
+ warn_or_error (G_("The STATUS specified in OPEN statement at %L "
"cannot have the value SCRATCH if a FILE specifier "
- "is present");
+ "is present"), &open->status->where);
+ }
+ }
+
+ /* Checks on NEWUNIT specifier. */
+ if (open->newunit)
+ {
+ if (open->unit)
+ {
+ gfc_error ("UNIT specifier not allowed with NEWUNIT at %L",
+ &open->newunit->where);
+ return false;
+ }
+
+ if (!open->file &&
+ (!open->status ||
+ (open->status->expr_type == EXPR_CONSTANT
+ && gfc_wide_strncasecmp (open->status->value.character.string,
+ "scratch", 7) != 0)))
+ {
+ gfc_error ("NEWUNIT specifier must have FILE= "
+ "or STATUS='scratch' at %L", &open->newunit->where);
+ return false;
}
}
+ else if (!open->unit)
+ {
+ gfc_error ("OPEN statement at %L must have UNIT or NEWUNIT specified",
+ where);
+ return false;
+ }
/* Things that are not allowed for unformatted I/O. */
if (open->form && open->form->expr_type == EXPR_CONSTANT
&& gfc_wide_strncasecmp (open->form->value.character.string,
"unformatted", 11) == 0)
{
- const char *spec = (open->delim ? "DELIM "
- : (open->pad ? "PAD " : open->blank
- ? "BLANK " : ""));
+ locus *loc;
+ const char *spec;
+ if (open->delim)
+ {
+ loc = &open->delim->where;
+ spec = "DELIM ";
+ }
+ else if (open->pad)
+ {
+ loc = &open->pad->where;
+ spec = "PAD ";
+ }
+ else if (open->blank)
+ {
+ loc = &open->blank->where;
+ spec = "BLANK ";
+ }
+ else
+ {
+ loc = where;
+ spec = "";
+ }
+
+ warn_or_error (G_("%s specifier at %L not allowed in OPEN statement for "
+ "unformatted I/O"), spec, loc);
+ }
+
+ if (open->recl && open->access && open->access->expr_type == EXPR_CONSTANT
+ && gfc_wide_strncasecmp (open->access->value.character.string,
+ "stream", 6) == 0)
+ {
+ warn_or_error (G_("RECL specifier not allowed in OPEN statement at %L for "
+ "stream I/O"), &open->recl->where);
+ }
+
+ if (open->position
+ && open->access && open->access->expr_type == EXPR_CONSTANT
+ && !(gfc_wide_strncasecmp (open->access->value.character.string,
+ "sequential", 10) == 0
+ || gfc_wide_strncasecmp (open->access->value.character.string,
+ "stream", 6) == 0
+ || gfc_wide_strncasecmp (open->access->value.character.string,
+ "append", 6) == 0))
+ {
+ warn_or_error (G_("POSITION specifier in OPEN statement at %L only allowed "
+ "for stream or sequential ACCESS"), &open->position->where);
+ }
+
+ return true;
+#undef warn_or_error
+}
+
+
+/* Match an OPEN statement. */
+
+match
+gfc_match_open (void)
+{
+ gfc_open *open;
+ match m;
+
+ m = gfc_match_char ('(');
+ if (m == MATCH_NO)
+ return m;
+
+ open = XCNEW (gfc_open);
+
+ m = match_open_element (open);
+
+ if (m == MATCH_ERROR)
+ goto cleanup;
+ if (m == MATCH_NO)
+ {
+ m = gfc_match_expr (&open->unit);
+ if (m == MATCH_ERROR)
+ goto cleanup;
+ }
+
+ for (;;)
+ {
+ if (gfc_match_char (')') == MATCH_YES)
+ break;
+ if (gfc_match_char (',') != MATCH_YES)
+ goto syntax;
- warn_or_error ("%s specifier at %C not allowed in OPEN statement for "
- "unformatted I/O", spec);
+ m = match_open_element (open);
+ if (m == MATCH_ERROR)
+ goto cleanup;
+ if (m == MATCH_NO)
+ goto syntax;
}
- if (open->recl && open->access && open->access->expr_type == EXPR_CONSTANT
- && gfc_wide_strncasecmp (open->access->value.character.string,
- "stream", 6) == 0)
- {
- warn_or_error ("RECL specifier not allowed in OPEN statement at %C for "
- "stream I/O");
- }
+ if (gfc_match_eos () == MATCH_NO)
+ goto syntax;
- if (open->position
- && open->access && open->access->expr_type == EXPR_CONSTANT
- && !(gfc_wide_strncasecmp (open->access->value.character.string,
- "sequential", 10) == 0
- || gfc_wide_strncasecmp (open->access->value.character.string,
- "stream", 6) == 0
- || gfc_wide_strncasecmp (open->access->value.character.string,
- "append", 6) == 0))
+ if (gfc_pure (NULL))
{
- warn_or_error ("POSITION specifier in OPEN statement at %C only allowed "
- "for stream or sequential ACCESS");
+ gfc_error ("OPEN statement not allowed in PURE procedure at %C");
+ goto cleanup;
}
-#undef warn_or_error
+ gfc_unset_implicit_pure (NULL);
new_st.op = EXEC_OPEN;
new_st.ext.open = open;
if (m != MATCH_NO)
return m;
m = match_etag (&tag_iomsg, &close->iomsg);
- if (m == MATCH_YES && !check_char_variable (close->iomsg))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
m = match_out_tag (&tag_iostat, &close->iostat);
{
gfc_close *close;
match m;
- bool warn;
m = gfc_match_char ('(');
if (m == MATCH_NO)
gfc_unset_implicit_pure (NULL);
- warn = (close->iostat || close->err) ? true : false;
-
- /* Checks on the STATUS specifier. */
- if (close->status && close->status->expr_type == EXPR_CONSTANT)
- {
- static const char *status[] = { "KEEP", "DELETE", NULL };
-
- if (!is_char_type ("STATUS", close->status))
- goto cleanup;
-
- if (!compare_to_allowed_values ("STATUS", status, NULL, NULL,
- close->status->value.character.string,
- "CLOSE", warn))
- goto cleanup;
- }
-
new_st.op = EXEC_CLOSE;
new_st.ext.close = close;
return MATCH_YES;
}
-/* Resolve everything in a gfc_close structure. */
-
-bool
-gfc_resolve_close (gfc_close *close)
+static bool
+check_close_constraints (gfc_close *close, locus *where)
{
- RESOLVE_TAG (&tag_unit, close->unit);
- RESOLVE_TAG (&tag_iomsg, close->iomsg);
- RESOLVE_TAG (&tag_iostat, close->iostat);
- RESOLVE_TAG (&tag_status, close->status);
-
- if (!gfc_reference_st_label (close->err, ST_LABEL_TARGET))
- return false;
+ bool warn = (close->iostat || close->err) ? true : false;
if (close->unit == NULL)
{
- /* Find a locus from one of the arguments to close, when UNIT is
- not specified. */
- locus loc = gfc_current_locus;
- if (close->status)
- loc = close->status->where;
- else if (close->iostat)
- loc = close->iostat->where;
- else if (close->iomsg)
- loc = close->iomsg->where;
- else if (close->err)
- loc = close->err->where;
-
- gfc_error ("CLOSE statement at %L requires a UNIT number", &loc);
+ gfc_error ("CLOSE statement at %L requires a UNIT number", where);
return false;
}
&close->unit->where);
}
+ /* Checks on the STATUS specifier. */
+ if (close->status && close->status->expr_type == EXPR_CONSTANT)
+ {
+ static const char *status[] = { "KEEP", "DELETE", NULL };
+
+ if (!compare_to_allowed_values ("STATUS", status, NULL, NULL,
+ close->status->value.character.string,
+ "CLOSE", warn, &close->status->where))
+ return false;
+ }
+
return true;
}
+/* Resolve everything in a gfc_close structure. */
+
+bool
+gfc_resolve_close (gfc_close *close, locus *where)
+{
+ RESOLVE_TAG (&tag_unit, close->unit);
+ RESOLVE_TAG (&tag_iomsg, close->iomsg);
+ RESOLVE_TAG (&tag_iostat, close->iostat);
+ RESOLVE_TAG (&tag_status, close->status);
+
+ if (!gfc_reference_st_label (close->err, ST_LABEL_TARGET))
+ return false;
+
+ return check_close_constraints (close, where);
+}
+
/* Free a gfc_filepos structure. */
if (m != MATCH_NO)
return m;
m = match_etag (&tag_iomsg, &fp->iomsg);
- if (m == MATCH_YES && !check_char_variable (fp->iomsg))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
m = match_out_tag (&tag_iostat, &fp->iostat);
m = match_file_element (fp);
if (m == MATCH_ERROR)
- goto done;
+ goto cleanup;
if (m == MATCH_NO)
{
m = gfc_match_expr (&fp->unit);
bool
-gfc_resolve_filepos (gfc_filepos *fp)
+gfc_resolve_filepos (gfc_filepos *fp, locus *where)
{
RESOLVE_TAG (&tag_unit, fp->unit);
RESOLVE_TAG (&tag_iostat, fp->iostat);
RESOLVE_TAG (&tag_iomsg, fp->iomsg);
- if (!gfc_reference_st_label (fp->err, ST_LABEL_TARGET))
- return false;
- if (!fp->unit && (fp->iostat || fp->iomsg))
+ if (!fp->unit && (fp->iostat || fp->iomsg || fp->err))
{
- locus where;
- where = fp->iostat ? fp->iostat->where : fp->iomsg->where;
- gfc_error ("UNIT number missing in statement at %L", &where);
+ gfc_error ("UNIT number missing in statement at %L", where);
return false;
}
+ if (!gfc_reference_st_label (fp->err, ST_LABEL_TARGET))
+ return false;
+
if (fp->unit->expr_type == EXPR_CONSTANT
&& fp->unit->ts.type == BT_INTEGER
&& mpz_sgn (fp->unit->value.integer) < 0)
}
m = match_etag (&tag_e_async, &dt->asynchronous);
- if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", dt->asynchronous))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
m = match_etag (&tag_e_blank, &dt->blank);
if (m != MATCH_NO)
return m;
m = match_etag (&tag_iomsg, &dt->iomsg);
- if (m == MATCH_YES && !check_char_variable (dt->iomsg))
- return MATCH_ERROR;
if (m != MATCH_NO)
return m;
}
+static const char *
+io_kind_name (io_kind k);
+
+static bool
+check_io_constraints (io_kind k, gfc_dt *dt, gfc_code *io_code,
+ locus *spec_end);
+
/* Resolve everything in a gfc_dt structure. */
bool
-gfc_resolve_dt (gfc_dt *dt, locus *loc)
+gfc_resolve_dt (gfc_code *dt_code, gfc_dt *dt, locus *loc)
{
gfc_expr *e;
io_kind k;
RESOLVE_TAG (&tag_e_decimal, dt->decimal);
RESOLVE_TAG (&tag_e_async, dt->asynchronous);
+ /* Check I/O constraints.
+ To validate NAMELIST we need to check if we were also given an I/O list,
+ which is stored in code->block->next with op EXEC_TRANSFER.
+ Note that the I/O list was already resolved from resolve_transfer. */
+ gfc_code *io_code = NULL;
+ if (dt_code && dt_code->block && dt_code->block->next
+ && dt_code->block->next->op == EXEC_TRANSFER)
+ io_code = dt_code->block->next;
+
+ if (!check_io_constraints (k, dt, io_code, loc))
+ return false;
+
e = dt->io_unit;
if (e == NULL)
{
return false;
}
+ if (e->symtree && e->symtree->n.sym->attr.flavor == FL_PARAMETER
+ && e->ts.type == BT_CHARACTER)
+ {
+ gfc_error ("UNIT specification at %L must "
+ "not be a character PARAMETER", &e->where);
+ return false;
+ }
+
if (gfc_resolve_expr (e)
&& (e->ts.type != BT_INTEGER
&& (e->ts.type != BT_CHARACTER || e->expr_type != EXPR_VARIABLE)))
{
m = gfc_match_variable (&expr, 0);
if (m == MATCH_NO)
- gfc_error ("Expected variable in READ statement at %C");
+ {
+ gfc_error ("Expecting variable in READ statement at %C");
+ m = MATCH_ERROR;
+ }
+
+ if (m == MATCH_YES && expr->expr_type == EXPR_CONSTANT)
+ {
+ gfc_error ("Expecting variable or io-implied-do in READ statement "
+ "at %L", &expr->where);
+ m = MATCH_ERROR;
+ }
+
+ if (m == MATCH_YES
+ && expr->expr_type == EXPR_VARIABLE
+ && expr->symtree->n.sym->attr.external)
+ {
+ gfc_error ("Expecting variable or io-implied-do at %L",
+ &expr->where);
+ m = MATCH_ERROR;
+ }
}
else
{
if (m == MATCH_NO)
gfc_error ("Expected expression in %s statement at %C",
io_kind_name (k));
+
+ if (m == MATCH_YES && expr->ts.type == BT_BOZ)
+ {
+ if (gfc_invalid_boz (G_("BOZ literal constant at %L cannot appear in"
+ " an output IO list"), &gfc_current_locus))
+ return MATCH_ERROR;
+ if (!gfc_boz2int (expr, gfc_max_integer_kind))
+ return MATCH_ERROR;
+ };
}
if (m == MATCH_YES && k == M_READ && gfc_check_do_variable (expr->symtree))
/* Check the constraints for a data transfer statement. The majority of the
- constraints appearing in 9.4 of the standard appear here. Some are handled
- in resolve_tag and others in gfc_resolve_dt. Also set the async_io_dt flag
- and, if necessary, the asynchronous flag on the SIZE argument. */
+ constraints appearing in 9.4 of the standard appear here.
-static match
+ Tag expressions are already resolved by resolve_tag, which includes
+ verifying the type, that they are scalar, and verifying that BT_CHARACTER
+ tags are of default kind. */
+
+static bool
check_io_constraints (io_kind k, gfc_dt *dt, gfc_code *io_code,
locus *spec_end)
{
-#define io_constraint(condition,msg,arg)\
+#define io_constraint(condition, msg, arg)\
if (condition) \
{\
- gfc_error(msg,arg);\
- m = MATCH_ERROR;\
+ if ((arg)->lb != NULL)\
+ gfc_error ((msg), (arg));\
+ else\
+ gfc_error ((msg), spec_end);\
+ return false;\
}
- match m;
gfc_expr *expr;
gfc_symbol *sym = NULL;
bool warn, unformatted;
unformatted = dt->format_expr == NULL && dt->format_label == NULL
&& dt->namelist == NULL;
- m = MATCH_YES;
-
expr = dt->io_unit;
if (expr && expr->expr_type == EXPR_VARIABLE
&& expr->ts.type == BT_CHARACTER)
io_constraint (dt->rec != NULL,
"REC tag at %L is incompatible with internal file",
&dt->rec->where);
-
+
io_constraint (dt->pos != NULL,
"POS tag at %L is incompatible with internal file",
&dt->pos->where);
{
if (!gfc_notify_std (GFC_STD_F2003, "Internal file at %L with "
"namelist", &expr->where))
- m = MATCH_ERROR;
+ return false;
}
io_constraint (dt->advance != NULL,
if (expr && expr->ts.type != BT_CHARACTER)
{
- io_constraint (gfc_pure (NULL) && (k == M_READ || k == M_WRITE),
- "IO UNIT in %s statement at %C must be "
+ if (gfc_pure (NULL) && (k == M_READ || k == M_WRITE))
+ {
+ gfc_error ("IO UNIT in %s statement at %L must be "
"an internal file in a PURE procedure",
- io_kind_name (k));
+ io_kind_name (k), &expr->where);
+ return false;
+ }
if (k == M_READ || k == M_WRITE)
gfc_unset_implicit_pure (NULL);
}
- if (k != M_READ)
- {
- io_constraint (dt->end, "END tag not allowed with output at %L",
- &dt->end_where);
-
- io_constraint (dt->eor, "EOR tag not allowed with output at %L",
- &dt->eor_where);
-
- io_constraint (dt->blank, "BLANK= specifier not allowed with output at %L",
- &dt->blank->where);
-
- io_constraint (dt->pad, "PAD= specifier not allowed with output at %L",
- &dt->pad->where);
-
- io_constraint (dt->size, "SIZE= specifier not allowed with output at %L",
- &dt->size->where);
- }
- else
- {
- io_constraint (dt->size && dt->advance == NULL,
- "SIZE tag at %L requires an ADVANCE tag",
- &dt->size->where);
-
- io_constraint (dt->eor && dt->advance == NULL,
- "EOR tag at %L requires an ADVANCE tag",
- &dt->eor_where);
- }
-
- if (dt->asynchronous)
+ if (dt->asynchronous)
{
- int num;
+ int num = -1;
static const char * asynchronous[] = { "YES", "NO", NULL };
+ /* Note: gfc_reduce_init_expr reports an error if not init-expr. */
if (!gfc_reduce_init_expr (dt->asynchronous))
- {
- gfc_error ("ASYNCHRONOUS= specifier at %L must be an initialization "
- "expression", &dt->asynchronous->where);
- return MATCH_ERROR;
- }
-
- if (!is_char_type ("ASYNCHRONOUS", dt->asynchronous))
- return MATCH_ERROR;
+ return false;
if (!compare_to_allowed_values
("ASYNCHRONOUS", asynchronous, NULL, NULL,
dt->asynchronous->value.character.string,
- io_kind_name (k), warn, &num))
- return MATCH_ERROR;
+ io_kind_name (k), warn, &dt->asynchronous->where, &num))
+ return false;
+
+ gcc_checking_assert (num != -1);
+
+ /* For "YES", mark related symbols as asynchronous. */
+ if (num == 0)
+ {
+ /* SIZE variable. */
+ if (dt->size)
+ dt->size->symtree->n.sym->attr.asynchronous = 1;
+
+ /* Variables in a NAMELIST. */
+ if (dt->namelist)
+ for (gfc_namelist *nl = dt->namelist->namelist; nl; nl = nl->next)
+ nl->sym->attr.asynchronous = 1;
+
+ /* Variables in an I/O list. */
+ for (gfc_code *xfer = io_code; xfer && xfer->op == EXEC_TRANSFER;
+ xfer = xfer->next)
+ {
+ gfc_expr *expr = xfer->expr1;
+ while (expr != NULL && expr->expr_type == EXPR_OP
+ && expr->value.op.op == INTRINSIC_PARENTHESES)
+ expr = expr->value.op.op1;
- /* Best to put this here because the yes/no info is still around. */
- async_io_dt = num == 0;
- if (async_io_dt && dt->size)
- dt->size->symtree->n.sym->attr.asynchronous = 1;
+ if (expr && expr->expr_type == EXPR_VARIABLE)
+ expr->symtree->n.sym->attr.asynchronous = 1;
+ }
+ }
}
- else
- async_io_dt = false;
if (dt->id)
{
if (dt->decimal)
{
- if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %C "
- "not allowed in Fortran 95"))
- return MATCH_ERROR;
+ if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %L "
+ "not allowed in Fortran 95", &dt->decimal->where))
+ return false;
if (dt->decimal->expr_type == EXPR_CONSTANT)
{
static const char * decimal[] = { "COMMA", "POINT", NULL };
- if (!is_char_type ("DECIMAL", dt->decimal))
- return MATCH_ERROR;
-
if (!compare_to_allowed_values ("DECIMAL", decimal, NULL, NULL,
dt->decimal->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn,
+ &dt->decimal->where))
+ return false;
io_constraint (unformatted,
"the DECIMAL= specifier at %L must be with an "
"explicit format expression", &dt->decimal->where);
}
}
-
+
if (dt->blank)
{
- if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %C "
- "not allowed in Fortran 95"))
- return MATCH_ERROR;
-
- if (!is_char_type ("BLANK", dt->blank))
- return MATCH_ERROR;
+ if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %L "
+ "not allowed in Fortran 95", &dt->blank->where))
+ return false;
if (dt->blank->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("BLANK", blank, NULL, NULL,
dt->blank->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn,
+ &dt->blank->where))
+ return false;
io_constraint (unformatted,
"the BLANK= specifier at %L must be with an "
if (dt->pad)
{
- if (!gfc_notify_std (GFC_STD_F2003, "PAD= at %C "
- "not allowed in Fortran 95"))
- return MATCH_ERROR;
-
- if (!is_char_type ("PAD", dt->pad))
- return MATCH_ERROR;
+ if (!gfc_notify_std (GFC_STD_F2003, "PAD= at %L "
+ "not allowed in Fortran 95", &dt->pad->where))
+ return false;
if (dt->pad->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("PAD", pad, NULL, NULL,
dt->pad->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn,
+ &dt->pad->where))
+ return false;
io_constraint (unformatted,
"the PAD= specifier at %L must be with an "
if (dt->round)
{
- if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %C "
- "not allowed in Fortran 95"))
- return MATCH_ERROR;
-
- if (!is_char_type ("ROUND", dt->round))
- return MATCH_ERROR;
+ if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %L "
+ "not allowed in Fortran 95", &dt->round->where))
+ return false;
if (dt->round->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("ROUND", round, NULL, NULL,
dt->round->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn,
+ &dt->round->where))
+ return false;
}
}
-
+
if (dt->sign)
{
/* When implemented, change the following to use gfc_notify_std F2003.
- if (gfc_notify_std (GFC_STD_F2003, "SIGN= at %C "
- "not allowed in Fortran 95") == false)
- return MATCH_ERROR; */
-
- if (!is_char_type ("SIGN", dt->sign))
- return MATCH_ERROR;
+ if (gfc_notify_std (GFC_STD_F2003, "SIGN= at %L "
+ "not allowed in Fortran 95", &dt->sign->where) == false)
+ return false; */
if (dt->sign->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("SIGN", sign, NULL, NULL,
dt->sign->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn, &dt->sign->where))
+ return false;
io_constraint (unformatted,
"SIGN= specifier at %L must be with an "
if (dt->delim)
{
- if (!gfc_notify_std (GFC_STD_F2003, "DELIM= at %C "
- "not allowed in Fortran 95"))
- return MATCH_ERROR;
-
- if (!is_char_type ("DELIM", dt->delim))
- return MATCH_ERROR;
+ if (!gfc_notify_std (GFC_STD_F2003, "DELIM= at %L "
+ "not allowed in Fortran 95", &dt->delim->where))
+ return false;
if (dt->delim->expr_type == EXPR_CONSTANT)
{
if (!compare_to_allowed_values ("DELIM", delim, NULL, NULL,
dt->delim->value.character.string,
- io_kind_name (k), warn))
- return MATCH_ERROR;
+ io_kind_name (k), warn,
+ &dt->delim->where))
+ return false;
io_constraint (k == M_READ,
"DELIM= specifier at %L not allowed in a "
"READ statement", &dt->delim->where);
-
+
io_constraint (dt->format_label != &format_asterisk
&& dt->namelist == NULL,
"DELIM= specifier at %L must have FMT=*",
"NML= specifier", &dt->delim->where);
}
}
-
+
if (dt->namelist)
{
io_constraint (io_code && dt->namelist,
io_constraint (dt->eor && not_no && k == M_READ,
"EOR tag at %L requires an ADVANCE = %<NO%>",
- &dt->eor_where);
+ &dt->eor_where);
}
- expr = dt->format_expr;
- if (!gfc_simplify_expr (expr, 0)
- || !check_format_string (expr, k == M_READ))
- return MATCH_ERROR;
+ if (k != M_READ)
+ {
+ io_constraint (dt->end, "END tag not allowed with output at %L",
+ &dt->end_where);
- return m;
-}
+ io_constraint (dt->eor, "EOR tag not allowed with output at %L",
+ &dt->eor_where);
+
+ io_constraint (dt->blank,
+ "BLANK= specifier not allowed with output at %L",
+ &dt->blank->where);
+
+ io_constraint (dt->pad, "PAD= specifier not allowed with output at %L",
+ &dt->pad->where);
+
+ io_constraint (dt->size, "SIZE= specifier not allowed with output at %L",
+ &dt->size->where);
+ }
+ else
+ {
+ io_constraint (dt->size && dt->advance == NULL,
+ "SIZE tag at %L requires an ADVANCE tag",
+ &dt->size->where);
+
+ io_constraint (dt->eor && dt->advance == NULL,
+ "EOR tag at %L requires an ADVANCE tag",
+ &dt->eor_where);
+ }
+
+ return true;
#undef io_constraint
+}
/* Match a READ, WRITE or PRINT statement. */
gfc_symbol *sym;
int comma_flag;
locus where;
- locus spec_end, control;
+ locus control;
gfc_dt *dt;
match m;
else
gfc_current_locus = where;
}
+
+ if (gfc_match_char ('*') == MATCH_YES
+ && gfc_match_char(',') == MATCH_YES)
+ {
+ locus where2 = gfc_current_locus;
+ if (gfc_match_eos () == MATCH_YES)
+ {
+ gfc_current_locus = where2;
+ gfc_error ("Comma after * at %C not allowed without I/O list");
+ m = MATCH_ERROR;
+ goto cleanup;
+ }
+ else
+ gfc_current_locus = where;
+ }
+ else
+ gfc_current_locus = where;
}
if (gfc_current_form == FORM_FREE)
get_io_list:
- /* Used in check_io_constraints, where no locus is available. */
- spec_end = gfc_current_locus;
-
/* Save the IO kind for later use. */
dt->dt_io_kind = gfc_get_iokind_expr (&gfc_current_locus, k);
goto syntax;
}
- /* See if we want to use defaults for missing exponents in real transfers. */
- if (flag_dec)
- dt->default_exp = 1;
-
- /* A full IO statement has been matched. Check the constraints. spec_end is
- supplied for cases where no locus is supplied. */
- m = check_io_constraints (k, dt, io_code, &spec_end);
+ /* See if we want to use defaults for missing exponents in real transfers
+ and other DEC runtime extensions. */
+ if (flag_dec_format_defaults)
+ dt->dec_ext = 1;
- if (m == MATCH_ERROR)
- goto cleanup;
+ /* Check the format string now. */
+ if (dt->format_expr
+ && (!gfc_simplify_expr (dt->format_expr, 0)
+ || !check_format_string (dt->format_expr, k == M_READ)))
+ return MATCH_ERROR;
new_st.op = (k == M_READ) ? EXEC_READ : EXEC_WRITE;
new_st.ext.dt = dt;
RETM m = match_etag (&tag_file, &inquire->file);
RETM m = match_ltag (&tag_err, &inquire->err);
RETM m = match_etag (&tag_iomsg, &inquire->iomsg);
- if (m == MATCH_YES && !check_char_variable (inquire->iomsg))
- return MATCH_ERROR;
RETM m = match_out_tag (&tag_iostat, &inquire->iostat);
RETM m = match_vtag (&tag_exist, &inquire->exist);
RETM m = match_vtag (&tag_opened, &inquire->opened);
RETM m = match_vtag (&tag_write, &inquire->write);
RETM m = match_vtag (&tag_readwrite, &inquire->readwrite);
RETM m = match_vtag (&tag_s_async, &inquire->asynchronous);
- if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", inquire->asynchronous))
- return MATCH_ERROR;
RETM m = match_vtag (&tag_s_delim, &inquire->delim);
RETM m = match_vtag (&tag_s_decimal, &inquire->decimal);
RETM m = match_out_tag (&tag_size, &inquire->size);
if (m == MATCH_NO)
goto syntax;
+ for (gfc_code *c = code; c; c = c->next)
+ if (c->expr1 && c->expr1->expr_type == EXPR_FUNCTION
+ && c->expr1->symtree && c->expr1->symtree->n.sym->attr.function
+ && !c->expr1->symtree->n.sym->attr.external
+ && strcmp (c->expr1->symtree->name, "null") == 0)
+ {
+ gfc_error ("NULL() near %L cannot appear in INQUIRE statement",
+ &c->expr1->where);
+ goto cleanup;
+ }
+
new_st.op = EXEC_IOLENGTH;
new_st.expr1 = inquire->iolength;
new_st.ext.inquire = inquire;
&& ((mpz_get_si (inquire->unit->value.integer) == GFC_INTERNAL_UNIT4)
|| (mpz_get_si (inquire->unit->value.integer) == GFC_INTERNAL_UNIT)))
{
- gfc_error ("UNIT number in INQUIRE statement at %L can not "
+ gfc_error ("UNIT number in INQUIRE statement at %L cannot "
"be %d", &loc, (int) mpz_get_si (inquire->unit->value.integer));
goto cleanup;
}
RETM m = match_ltag (&tag_end, &wait->end);
RETM m = match_ltag (&tag_eor, &wait->eor);
RETM m = match_etag (&tag_iomsg, &wait->iomsg);
- if (m == MATCH_YES && !check_char_variable (wait->iomsg))
- return MATCH_ERROR;
RETM m = match_out_tag (&tag_iostat, &wait->iostat);
RETM m = match_etag (&tag_id, &wait->id);
RETM return MATCH_NO;