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]
Other format: [Raw text]

Re: share all int csts


Nathan Sidwell wrote:

This patch implements a hash table to back the INT_CST vector caches.
When an integer is small, it is still placed in the type's vector cache.
When it is large, it is placed in this hash table.

There was no difference in unchecked bootstrap time on darwin.
It shows a small speedup ~0.2% on qt and similarly a small increase
for geralds PR 8361 testcase.

booted & tested on i686-pc-linux-gnu, ok?

Yes, assuming that constants with TREE_OVERFLOW don't end up in there. (I see the hash function doesn't check for that.) The hard-coded 1024 for the initial hash table size isn't very attractive, but it doesn't seem very --param-worthy, either.


--
Mark Mitchell
CodeSourcery, LLC
(916) 791-8304
mark@codesourcery.com


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