This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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][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


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