This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils 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: Optimize IA64 brl to br

From: Jim Wilson <wilson at specifixinc dot com>
To: "H. J. Lu" <hjl at lucon dot org>
Cc: Richard Henderson <rth at redhat dot com>,Zack Weinberg <zack at codesourcery dot com>, binutils at sources dot redhat dot com
Date: 30 Jan 2004 12:24:19 -0800
Subject: Re: PATCH: Optimize IA64 brl to br
References: <20040129000632.GA14782@lucon.org><87isiv7gss.fsf@egil.codesourcery.com> <20040129170735.GA31422@lucon.org><1075428976.1011.82.camel@leaf.tuliptree.org><20040130030933.GA12268@redhat.com><1075443815.1231.99.camel@leaf.tuliptree.org> <20040130194512.GA25041@lucon.org>

On Fri, 2004-01-30 at 11:45, H. J. Lu wrote:
> 	* elfxx-ia64.c (elfNN_ia64_relax_section): Optimize brl to br
> 	during the relax finalize pass.

OK.

You didn't move the brl->br relaxation code into a subroutine as I
suggested.  Everything from hit_addr to bfd_put_64 could be in a
subroutine.  I think that would help make the code more readable.  This
doesn't seem to be a big deal though.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com

Follow-Ups:
- Re: PATCH: Optimize IA64 brl to br
  - From: H. J. Lu

References:
- PATCH: Optimize IA64 brl to br
  - From: H. J. Lu
- Re: PATCH: Optimize IA64 brl to br
  - From: Zack Weinberg
- Re: PATCH: Optimize IA64 brl to br
  - From: H. J. Lu
- Re: PATCH: Optimize IA64 brl to br
  - From: Jim Wilson
- Re: PATCH: Optimize IA64 brl to br
  - From: Richard Henderson
- Re: PATCH: Optimize IA64 brl to br
  - From: Jim Wilson
- Re: PATCH: Optimize IA64 brl to br
  - From: H. J. Lu

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