This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [gomp4, committed] Fix if-clause handling for kernels
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: "gcc-patches at gnu dot org" <gcc-patches at gnu dot org>, Jim Norris <James_Norris at mentor dot com>
- Date: Fri, 17 Jul 2015 20:47:14 +0200
- Subject: Re: [gomp4, committed] Fix if-clause handling for kernels
- Authentication-results: sourceware.org; auth=none
- References: <55A94B3A dot 4080200 at mentor dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Jul 17, 2015 at 08:36:42PM +0200, Tom de Vries wrote:
> diff --git a/gcc/omp-low.c b/gcc/omp-low.c
> index 0419dcd..9b062e6 100644
> --- a/gcc/omp-low.c
> +++ b/gcc/omp-low.c
> @@ -9643,7 +9643,8 @@ expand_omp_target (struct omp_region *region)
> tree tmp_var;
>
> tmp_var = create_tmp_var (TREE_TYPE (device));
> - if (offloaded)
> + if (offloaded
> + && do_splitoff)
Such short conditions belong on a single line IMHO.
Jakub