Re: Drop generic32 cost model

2013-09-20 Thread Ian Lance Taylor
On Fri, Sep 20, 2013 at 6:00 AM, Zamyatin, Igor igor.zamya...@intel.com wrote: For x86 32-bit is still important due to mobile market so we would like to make some measurements first to see how this new generic model works on 32 bits I would expect people compiling code for the mobile

Re: Drop generic32 cost model

2013-09-20 Thread Jan Hubicka
On Fri, Sep 20, 2013 at 6:00 AM, Zamyatin, Igor igor.zamya...@intel.com wrote: For x86 32-bit is still important due to mobile market so we would like to make some measurements first to see how this new generic model works on 32 bits I assume by mobile market you mean atom CPU these

Re: Drop generic32 cost model

2013-09-19 Thread Ian Lance Taylor
On Wed, Sep 18, 2013 at 1:39 PM, Jan Hubicka hubi...@ucw.cz wrote: when generic model was introduced, the 32bit only CPUs was still common on the market. It would be stupid to tune 64bit code for CPUs that will never run it. We thus introduced two models - generic32 that was considering

Re: Drop generic32 cost model

2013-09-19 Thread Jan Hubicka
decide_alg is being called from ix86_expand_movmem, from expand_builtin_memcpy, for the call at line 61 of go-append.c. __builtin_memcpy (n, a.__values, a.__count * element_size); I'm continuing to look. Indeed it is problem of this patch - the issue is that generic64 had dummy

Re: Drop generic32 cost model

2013-09-19 Thread Ian Lance Taylor
On Thu, Sep 19, 2013 at 8:38 AM, Jan Hubicka hubi...@ucw.cz wrote: decide_alg is being called from ix86_expand_movmem, from expand_builtin_memcpy, for the call at line 61 of go-append.c. __builtin_memcpy (n, a.__values, a.__count * element_size); I'm continuing to look. Indeed it is

Re: Drop generic32 cost model

2013-09-18 Thread H.J. Lu
On Wed, Sep 18, 2013 at 1:39 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, when generic model was introduced, the 32bit only CPUs was still common on the market. It would be stupid to tune 64bit code for CPUs that will never run it. We thus introduced two models - generic32 that was considering