On 07/10/2011 02:33 AM, Uros Bizjak wrote:
> On Sun, Jul 10, 2011 at 3:34 AM, Richard Henderson <r...@redhat.com> wrote:
>> I developed this patch while working on the dwarf2 pass series.
>> This was before I bypassed the entire problem by removing the
>> !deep branch prediction paths.
>>
>> Ideally, we'd do this generically from gimple.  Less ideally,
>> but still better, is to always emit rtl, and support that in
>> the middle end without so many hacks in the back end.
> 
> Looks good to me!
> 
> +  reload_completed = 1;
> +  epilogue_completed = 1;
> 
> Do we really need these?

Dunno.  Copied them from ia64.  I'll run the testsuite again without.

> +       tmp_regno = CX_REG;
>         if ((ccvt & (IX86_CALLCVT_FASTCALL | IX86_CALLCVT_THISCALL)) != 0)
>           tmp_regno = AX_REG;
> 
> if (...)
>   tmp_regno = AX_REG;
> else
>   tmp_regno = CX_REG;

If you insist.


r~

Reply via email to