> Hi Jakub,
>
> Good catch! Does it fix any specific PR?
>
> If you don't have the time, I would be happy to apply the correction to
> 13-branch through to mainline.
I caught it with my WIP patch to improve tree-ssa-dce. I am not aware
it can produce wrong code. It will likely lead to missed o
> On Mon, 21 Nov 2022 20:02:49 +0100
> Jan Hubicka wrote:
>
> > > Hi Honza, Ping.
> > > Regtests cleanly for c,fortran,c++,ada,d,go,lto,objc,obj-c++
> > > Ok?
> > > I'd need this for attribute target_clones for the Fortran FE.
> > Sorry
> Hi Honza, Ping.
> Regtests cleanly for c,fortran,c++,ada,d,go,lto,objc,obj-c++
> Ok?
> I'd need this for attribute target_clones for the Fortran FE.
Sorry for delay here.
> > void
> > @@ -303,6 +301,10 @@ symbol_table::change_decl_assembler_name (tree decl,
> > tree name)
> > warning (0, "%
nd so only
> keep the inline copy.
I have updated the matmul as follows.
gcc/testsuite/ChangeLog:
2021-11-11 Jan Hubicka
* gfortran.dg/inline_matmul_17.f90: Fix template
diff --git a/gcc/testsuite/gfortran.dg/inline_matmul_17.f90
b/gcc/testsuite/gfortran.dg/inline_matmul_17.f90
>
> Fortran has for a long time 'character(len=5), allocatable" or
> "character(len=*)". In the first case, the "5" can be ignored as both
> caller and callee know the length. In the second case, the length is
> determined by the argument, but it cannot be changed.
>
> Since a not-that-short whil