This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


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

Re: htab_hash_string


Richard Henderson wrote:-

> You think it's worth an extra loop test for the count?
> I wouldn't have thought so.

I agree.  My thoughts on this probably come from the hashtable.c
implementation, where the length is readily available, and the loop
counter, and therefore truncatable.

> It isn't quite equivalent.

You're right, I'd forgotten.  I would still like to use your function
instead; I might patch that later.

> We should, however, get rid if hashtable.c altogether.

How / why?  Or do you mean just move it somewhere else?  It's more
specialised than the generic hashtables, and for a good reason.

Neil.


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