On Mon, 23 Apr 2018, Tom de Vries wrote:

> [ was: Re: [PATCH] Handle empty infinite loops in OpenACC for PR84955 ]
> 
> On 04/16/2018 08:13 PM, Tom de Vries wrote:
> > On 04/12/2018 08:58 PM, Jakub Jelinek wrote:
> > > On Thu, Apr 12, 2018 at 11:39:43AM -0700, Cesar Philippidis wrote:
> > > > Strange. I didn't observe any regressions when I tested it. But, then
> > > > again, I was testing against revision
> > > > 
> > > > r259092 | jason | 2018-04-04 09:42:55 -0700 (Wed, 04 Apr 2018) | 4 lines
> > > > 
> > > > which is over a week old. I'll revert that patch for now, and revisit
> > > > this issue in stage1.
> > > 
> > > You should have kept the omp-expand.c chunk, that is correct and shouldn't
> > > cause issues.
> > 
> > Committed as attached (with correct changelog entry, the previously
> > committed patch had an incorrect one).
> > 
> > I've filed the lto ICE (described as "the second problem" in this thread) as
> > PR85422 - "[openacc] ICE at cfgloop.c:468: segfault in flow_loops_find".
> > 
> 
> Hi,
> 
> your updated patch [ as posted here (
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85422#c9 ) ]  fixes the
> flow_loops_find segfault, without causing the regressions mentioned here (
> https://gcc.gnu.org/ml/gcc-patches/2018-04/msg00628.html ).
> 
> Build x86_64 with nvptx accelerator, ran libgomp testsuite.
> 
> Bootstrapped and reg-tested x86_64 with -m64/-m32.
> 
> OK for stage4/stage1 trunk?
OK for stage1.

Thanks,
Richard.

Reply via email to