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 Mon, 2013-01-14 at 21:00 +0530, 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.

I'm fine with this.

Ryan



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