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

Add trailing dots to fortran io fnspecs to match signature

2020-09-29 Thread Jan Hubicka
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. Bootstrapped/regtested x86_64-linux. OK? Honza *