This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [gomp4] correct a tile bug with combined loops in fortran

From: Nathan Sidwell <nathan at codesourcery dot com>
To: Cesar Philippidis <cesar at codesourcery dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Fortran List <fortran at gcc dot gnu dot org>
Date: Thu, 6 Oct 2016 06:16:47 -0400
Subject: Re: [gomp4] correct a tile bug with combined loops in fortran
Authentication-results: sourceware.org; auth=none
References: <b0310400-9891-dafd-8112-df45bf3e56ae@codesourcery.com>

On 10/05/16 23:14, Cesar Philippidis wrote:

Nathan spotted a bug where the fortran FE wasn't lowering the tile
clause properly in combined parallel/kernels loops. The problem was that
gfc_trans_omp_d has two sources of clauses, code->ext.omp_clauses and
do_clauses. Because of the way that the fortran FE splits combined
loops, the tile clause is only associated with do_clauses. This patch
teaches gfc_trans_omp_do to check for the tile clause in both code and
do_clauses.

I've applied this patch to gomp-4_0-branch.



thanks!


--
Nathan Sidwell

References:
- [gomp4] correct a tile bug with combined loops in fortran
  - From: Cesar Philippidis

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]