diff mbox

Drop generic32 cost model

Message ID 20130919153822.GB8296@atrey.karlin.mff.cuni.cz
State New
Headers show

Commit Message

Jan Hubicka Sept. 19, 2013, 3:38 p.m. UTC 
> 
> 
> 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 32bit alg entries (those should not cause infinite loop, I will
fix that too)

I am testing:

> 
> Ian

Comments

Ian Lance Taylor Sept. 19, 2013, 3:41 p.m. UTC | #1 
On Thu, Sep 19, 2013 at 8:38 AM, Jan Hubicka <hubicka@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 problem of this patch - the issue is that generic64 had
> dummy 32bit alg entries (those should not cause infinite loop, I will
> fix that too)

Thanks, that patch at least lets the build continue.

Ian
diff mbox

Patch

Index: i386.c
===================================================================
--- i386.c      (revision 202741)
+++ i386.c      (working copy)
@@ -1648,11 +1648,13 @@  struct processor_costs slm_cost = {
    and btver1 (and newer chips).  */
 
 static stringop_algs generic_memcpy[2] = {
-  DUMMY_STRINGOP_ALGS,
+  {libcall, {{32, loop, false}, {8192, rep_prefix_4_byte, false},
+             {-1, libcall, false}}},
   {libcall, {{32, loop, false}, {8192, rep_prefix_8_byte, false},
              {-1, libcall, false}}}};
 static stringop_algs generic_memset[2] = {
-  DUMMY_STRINGOP_ALGS,
+  {libcall, {{32, loop, false}, {8192, rep_prefix_4_byte, false},
+             {-1, libcall, false}}},
   {libcall, {{32, loop, false}, {8192, rep_prefix_8_byte, false},
              {-1, libcall, false}}}};
 static const