Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Richard Biener
On Tue, 29 Sep 2020, Jan Hubicka wrote: > > On September 29, 2020 4:20:42 PM GMT+02:00, Jan Hubicka > > 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 tr

Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Jan Hubicka
> On September 29, 2020 4:20:42 PM GMT+02:00, Jan Hubicka > 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 ha

Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Richard Biener
On September 29, 2020 4:20:42 PM GMT+02:00, Jan Hubicka 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 pa

Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Martin Liška
On 9/29/20 4:45 PM, Jan Hubicka wrote: My fixup is longer:) Heh. So please include my gcc_checking_assert hunk to your patch. Martin All strings starting with R or W are wrong. However I have instances of miamatched lengths say for caf_register, deregister and others. There are few cases wh

Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Jan Hubicka
> On 9/29/20 4:20 PM, Jan Hubicka wrote: > > 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 > > I'm sending the run-time sanity check patch and few more places that assert. > I'm going to test the patch

Re: Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Martin Liška
On 9/29/20 4:20 PM, Jan Hubicka wrote: 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 I'm sending the run-time sanity check patch and few more places that assert. I'm going to test the patch. Martin >Fr