Re: [PATCH 0/2] fortran: Ignore unused arguments for scalarisation [PR97896]

On 07.11.21 17:17, Mikael Morin via Fortran wrote: Regression-tested on x86_64-linux-gnu. Ok for master and 11 branch? OK. Just one remark: Since just reverting my old patch would introduce a regression for that one revision, please squash the patches before committing. Thanks a lot for th

Re: [PATCH v3 0/5] fortran: Ignore unused arguments for scalarisation [PR97896]

Hi Mikael, Regression-tested on x86_64-linux-gnu. Ok for master? This looks quite good, and is (I think) a cleaner version than what we had before. OK. Thanks a lot for the patch(es)! Best regards Thomas

Re: [Patch] Fortran/openmp: Fix '!$omp end'

On Thu, Nov 11, 2021 at 06:11:23PM +0100, Tobias Burnus wrote: > --- a/gcc/fortran/parse.c > +++ b/gcc/fortran/parse.c > @@ -915,15 +915,16 @@ decode_omp_directive (void) >matcho ("error", gfc_match_omp_error, ST_OMP_ERROR); >matcho ("end atomic", gfc_match_omp_eos_error, ST_OMP_END

Re: [PATCH] vect: Remove vec_outside/inside_cost fields

> > Now afunc writes to __var_5_mma only indirectly so I think it is correct > > that > > we optimize the conditional out. > > > > Easy fix would be to add -fno-ipa-modref, but perhaps someone with > > better understanding of Fortran would help me to improve the testcase so > > the calls to matmul

[Patch] Fortran/openmp: Fix '!$omp end'

Found this when looking at the num_teams patch – and when converting clauses-1.c to clauses-1.f90. OK? Tobias - Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank T

Re: [Patch] Fortran/openmp: Add support for 2 argument num_teams clause

On Thu, Nov 11, 2021 at 04:04:04PM +0100, Tobias Burnus wrote: > Just the Fortran FE work + Fortranized version for the C tests. > > Tobias > - > Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 > München; Gesellschaft mit beschränkter Haftung; Geschäf

[Patch] Fortran/openmp: Add support for 2 argument num_teams clause

Just the Fortran FE work + Fortranized version for the C tests. Tobias - Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München;

Re: [PATCH] vect: Remove vec_outside/inside_cost fields

On Thu, Nov 11, 2021 at 10:45 AM Jan Hubicka via Gcc-patches wrote: > > > > > > > > > I think the patch causes the following on x86_64-linux-gnu: > > > > FAIL: gfortran.dg/inline_matmul_17.f90 -O scan-tree-dump-times > > > > optimized "matmul_r4" 2 > > > > > > I get that failure even with d70