Re: Speedup and cleanup hash-table.h

2014-12-19 Thread Jan Hubicka
> On December 19, 2014 12:48:22 AM CET, Jan Hubicka wrote: > >Hi, > >this patch started as experiment moving hash_table_mod1 inline because > >it shows > >high in streaming profiles and it represents a branch-less code that is > >good > >to schedule to surrounding instructions. > >While looking at

Re: Speedup and cleanup hash-table.h

2014-12-19 Thread Richard Biener
On December 19, 2014 12:48:22 AM CET, Jan Hubicka wrote: >Hi, >this patch started as experiment moving hash_table_mod1 inline because >it shows >high in streaming profiles and it represents a branch-less code that is >good >to schedule to surrounding instructions. >While looking at hash-tab.h I ho

Re: Speedup and cleanup hash-table.h

2014-12-18 Thread Jan Hubicka
> Jan Hubicka writes: > > > Hi, > > this patch started as experiment moving hash_table_mod1 inline because it > > shows > > high in streaming profiles and it represents a branch-less code that is good > > to schedule to surrounding instructions. > > FWIW with a good modern hash function it shou

Re: Speedup and cleanup hash-table.h

2014-12-18 Thread Andi Kleen
Jan Hubicka writes: > Hi, > this patch started as experiment moving hash_table_mod1 inline because it > shows > high in streaming profiles and it represents a branch-less code that is good > to schedule to surrounding instructions. FWIW with a good modern hash function it shouldn't be needed to