Add trailing dots to fortran io fnspecs to match signature
Richard Biener
rguenther@suse.de
Tue Sep 29 15:13:08 GMT 2020
On September 29, 2020 4:20:42 PM GMT+02:00, Jan Hubicka <hubicka@ucw.cz> wrote:
>Hi,
>this patch is not needed but makes it possible to sanity check that
>fnspec match function signature. It turns out that there are quite few
>mistakes in that in trans-decl and one mistake here.
>Transfer_derived has additional parameters.
Hmm, omitting trailing dots was on purpose to make the string short (also consider varargs...). You can still sanity check the prefix, no?
>Bootstrapped/regtested x86_64-linux. OK?
>Honza
>
> * transe-io.c (gfc_build_io_library_fndecls): Add traling "." for
> fnspecs so the match number of parameters.
>diff --git a/gcc/fortran/trans-io.c b/gcc/fortran/trans-io.c
>index 21bdd5ef0d8..363cca51ef9 100644
>--- a/gcc/fortran/trans-io.c
>+++ b/gcc/fortran/trans-io.c
>@@ -328,86 +328,86 @@ gfc_build_io_library_fndecls (void)
>dt_parm_type = build_pointer_type (st_parameter[IOPARM_ptype_dt].type);
>
> iocall[IOCALL_X_INTEGER] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_integer")), ".wW",
>+ get_identifier (PREFIX("transfer_integer")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_INTEGER_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_integer_write")), ".wR",
>+ get_identifier (PREFIX("transfer_integer_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
> iocall[IOCALL_X_LOGICAL] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_logical")), ".wW",
>+ get_identifier (PREFIX("transfer_logical")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_LOGICAL_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_logical_write")), ".wR",
>+ get_identifier (PREFIX("transfer_logical_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_CHARACTER] = gfc_build_library_function_decl_with_spec
>(
>- get_identifier (PREFIX("transfer_character")), ".wW",
>+ get_identifier (PREFIX("transfer_character")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node,
>gfc_charlen_type_node);
>
>iocall[IOCALL_X_CHARACTER_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_character_write")), ".wR",
>+ get_identifier (PREFIX("transfer_character_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node,
>gfc_charlen_type_node);
>
>iocall[IOCALL_X_CHARACTER_WIDE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_character_wide")), ".wW",
>+ get_identifier (PREFIX("transfer_character_wide")), ".wW..",
> void_type_node, 4, dt_parm_type, pvoid_type_node,
> gfc_charlen_type_node, gfc_int4_type_node);
>
> iocall[IOCALL_X_CHARACTER_WIDE_WRITE] =
> gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_character_wide_write")), ".wR",
>+ get_identifier (PREFIX("transfer_character_wide_write")), ".wR..",
> void_type_node, 4, dt_parm_type, pvoid_type_node,
> gfc_charlen_type_node, gfc_int4_type_node);
>
> iocall[IOCALL_X_REAL] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_real")), ".wW",
>+ get_identifier (PREFIX("transfer_real")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_REAL_WRITE] = gfc_build_library_function_decl_with_spec
>(
>- get_identifier (PREFIX("transfer_real_write")), ".wR",
>+ get_identifier (PREFIX("transfer_real_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
> iocall[IOCALL_X_COMPLEX] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_complex")), ".wW",
>+ get_identifier (PREFIX("transfer_complex")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_COMPLEX_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_complex_write")), ".wR",
>+ get_identifier (PREFIX("transfer_complex_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
> /* Version for __float128. */
> iocall[IOCALL_X_REAL128] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_real128")), ".wW",
>+ get_identifier (PREFIX("transfer_real128")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_REAL128_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_real128_write")), ".wR",
>+ get_identifier (PREFIX("transfer_real128_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_COMPLEX128] = gfc_build_library_function_decl_with_spec
>(
>- get_identifier (PREFIX("transfer_complex128")), ".wW",
>+ get_identifier (PREFIX("transfer_complex128")), ".wW.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
>iocall[IOCALL_X_COMPLEX128_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_complex128_write")), ".wR",
>+ get_identifier (PREFIX("transfer_complex128_write")), ".wR.",
> void_type_node, 3, dt_parm_type, pvoid_type_node, gfc_int4_type_node);
>
> iocall[IOCALL_X_ARRAY] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_array")), ".ww",
>+ get_identifier (PREFIX("transfer_array")), ".ww..",
> void_type_node, 4, dt_parm_type, pvoid_type_node,
> integer_type_node, gfc_charlen_type_node);
>
>iocall[IOCALL_X_ARRAY_WRITE] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_array_write")), ".wr",
>+ get_identifier (PREFIX("transfer_array_write")), ".wr..",
> void_type_node, 4, dt_parm_type, pvoid_type_node,
> integer_type_node, gfc_charlen_type_node);
>
> iocall[IOCALL_X_DERIVED] = gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("transfer_derived")), ".wrR",
>- void_type_node, 2, dt_parm_type, pvoid_type_node, pchar_type_node);
>+ get_identifier (PREFIX("transfer_derived")), ".wr",
>+ void_type_node, 2, dt_parm_type, pvoid_type_node);
>
> /* Library entry points */
>
>@@ -475,18 +475,18 @@ gfc_build_io_library_fndecls (void)
> void_type_node, 1, dt_parm_type);
>
>iocall[IOCALL_SET_NML_VAL] = gfc_build_library_function_decl_with_spec
>(
>- get_identifier (PREFIX("st_set_nml_var")), ".w.R",
>+ get_identifier (PREFIX("st_set_nml_var")), ".w.R...",
> void_type_node, 6, dt_parm_type, pvoid_type_node, pvoid_type_node,
> gfc_int4_type_node, gfc_charlen_type_node, get_dtype_type_node());
>
>iocall[IOCALL_SET_NML_DTIO_VAL] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("st_set_nml_dtio_var")), ".w.R",
>+ get_identifier (PREFIX("st_set_nml_dtio_var")), ".w.R.....",
> void_type_node, 8, dt_parm_type, pvoid_type_node, pvoid_type_node,
> gfc_int4_type_node, gfc_charlen_type_node, get_dtype_type_node(),
> pvoid_type_node, pvoid_type_node);
>
>iocall[IOCALL_SET_NML_VAL_DIM] =
>gfc_build_library_function_decl_with_spec (
>- get_identifier (PREFIX("st_set_nml_var_dim")), ".w",
>+ get_identifier (PREFIX("st_set_nml_var_dim")), ".w....",
> void_type_node, 5, dt_parm_type, gfc_int4_type_node,
> gfc_array_index_type, gfc_array_index_type, gfc_array_index_type);
> }
More information about the Gcc-patches
mailing list