This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [PATCH] -fopenmp-simd LTO ICE fix (PR middle-end/67452)

From: Jeff Law <law at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <rguenther at suse dot de>, Jan Hubicka <hubicka at ucw dot cz>
Cc: gcc-patches at gcc dot gnu dot org, Ilya Verbin <iverbin at gmail dot com>
Date: Fri, 4 Sep 2015 14:33:14 -0600
Subject: Re: [PATCH] -fopenmp-simd LTO ICE fix (PR middle-end/67452)
Authentication-results: sourceware.org; auth=none
References: <20150904152727 dot GL1847 at tucnak dot redhat dot com>

On 09/04/2015 09:27 AM, Jakub Jelinek wrote:

Hi!

This patch is a fix for gomp4.1 branch ICEs on
FAIL libgomp.c/for-5.c
FAIL libgomp.c++/for-13.C
that can be reproduced with intelmic emul offloading even on the trunk
with the attached testcase.  The problem is that LTO streaming doesn't
really like earlier unreferenced VAR_DECLs to be streamed late during
output_function (from output_cfg).

The patch fixes it by clearing loop->simduid when optimizations removed
all IL uses of the var and remove_unused_locals is about to nuke it from
cfun->local_decls.

Ok for trunk/5.3 if it passes bootstrap/regtest?

2015-09-04  Jakub Jelinek  <jakub@redhat.com>

	PR middle-end/67452
	* tree-ssa-live.c: Include cfgloop.h.
	(remove_unused_locals): Clear loop->simduid if simduid is about
	to be removed from cfun->local_decls.

	* gcc.dg/lto/pr67452_0.c: New test.

OK.
jeff

References:
- [PATCH] -fopenmp-simd LTO ICE fix (PR middle-end/67452)
  - From: Jakub Jelinek

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