On Mon, Sep 9, 2019 at 4:52 PM Mark Eggleston <mark.eggles...@codethink.co.uk> wrote: > To work around these problems I added a new length field to gfc_typespec > to used to produce the name of a character type if the character length > structure is not present.
> The addition of the length field is a bit of kludge any pointers > regarding a better solution will be appreciated. Thanks for the patch, I agree that we should print character type names better. However, I'm not really happy with this approach. Requiring us to keep track of the character length in two places sounds like a recipe for confusing bugs. I don't really have a good solution thought out for this, but I think this should be solved somehow before committing the patch. Secondly, character lengths can be longer than what fits into int. In gfortran.h you'll find typedef HOST_WIDE_INT gfc_charlen_t; and then you should use gfc_mpz_get_hwi() instead of mpz_get_si(). And for the *printf() format string you should use HOST_WIDE_INT_PRINT_DEC. Thanks, -- Janne Blomqvist
