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~