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: update address taken: don't drop clobbers

From: Jeff Law <law at redhat dot com>
To: Marc Glisse <marc dot glisse at inria dot fr>, gcc-patches at gcc dot gnu dot org
Date: Mon, 30 Jun 2014 13:31:31 -0600
Subject: Re: update address taken: don't drop clobbers
Authentication-results: sourceware.org; auth=none
References: <alpine dot DEB dot 2 dot 02 dot 1406282350110 dot 31815 at stedding dot saclay dot inria dot fr>

On 06/28/14 16:33, Marc Glisse wrote:

In an earlier version of the patch, I was using
get_or_create_ssa_default_def (cfun, sym);
(I was reusing the same variable). This passed bootstrap+testsuite on
all languages except for ada. Indeed, the compiler wanted to coalesce
several SSA_NAMEs, including those new ones, in out-of-ssa, but
couldn't.

And that's what you need to delve deeper into. Why did it refuse tocoalesce?


As long as the lifetimes do not overlap, then coalescing should have worked.

jeff

References:
- update address taken: don't drop clobbers
  - From: Marc Glisse

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