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: use hashtab for pseudo op table

From: Nick Clifton <nickc at redhat dot com>
To: Zack Weinberg <zack at codesourcery dot com>, Ben Elliston <bje at au1 dot ibm dot com>
Cc: binutils at sources dot redhat dot com
Date: Thu, 05 May 2005 10:56:59 +0100
Subject: Re: PATCH: use hashtab for pseudo op table
References: <4271E891.5090202@au.ibm.com> <874qdqj90o.fsf@codesourcery.com>

Hi Ben, Hi Zack,

> Zack Weinberg wrote:

I do not think replacing gas/hash.c with hashtab.c is a good idea, for
reasons laid out (somewhat cursorily) in
<http://sources.redhat.com/ml/binutils/2005-04/msg00056.html>.  I can
expand if anyone wants to hear it.

I agree - Ben, what is your motivation for removing hash.c ?

I am partially concerned because there has been a recent surge of interest in improving the performance of gas, and it sounds like switching over to using the libiberty hashtab implementation is going to be a performance hit.

Cheers
  Nick

Follow-Ups:
- Re: PATCH: use hashtab for pseudo op table
  - From: Alan Modra

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