This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH][PPC] Remove unnecessary local variable mptwo
On Monday, January 14, 2013 21:00:36 Siddhesh Poyarekar wrote:
> On Thu, Jan 10, 2013 at 10:06:39AM -0800, Roland McGrath wrote:
> > It seems appropriate to check. In many cases an initialized local
> > like that gets folded into its uses and occupies no stack space and
> > uses less code compared to the load from a global.
>
> Is this OK to commit? I verified that the local variable is
> redundant, i.e. it does not make any difference to the generated code,
> which just uses the global mptwo.
In that case it'S fine to commit,
Andreas
--
Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126