Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-11 Thread Uros Bizjak
On Sun, Jul 10, 2011 at 11:15 PM, Richard Henderson wrote: >> +  reload_completed = 1; >> +  epilogue_completed = 1; >> >> Do we really need these? Perhaps a comment should be added here, it is >> not obvious at the first sight... > > We didn't need these.  ia64 did because there we run a split p

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-10 Thread Richard Henderson
On 07/10/2011 02:33 AM, Uros Bizjak wrote: > + reload_completed = 1; > + epilogue_completed = 1; > > Do we really need these? Perhaps a comment should be added here, it is > not obvious at the first sight... We didn't need these. ia64 did because there we run a split pass to properly generate

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-10 Thread Jan Hubicka
> 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

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-10 Thread Richard Henderson
On 07/10/2011 02:33 AM, Uros Bizjak wrote: > On Sun, Jul 10, 2011 at 3:34 AM, Richard Henderson 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 g

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-10 Thread Uros Bizjak
On Sun, Jul 10, 2011 at 3:34 AM, Richard Henderson 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

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-09 Thread H.J. Lu
On Sat, Jul 9, 2011 at 6:43 PM, Richard Henderson wrote: > On 07/09/2011 06:42 PM, H.J. Lu wrote: >> On Sat, Jul 9, 2011 at 6:34 PM, Richard Henderson wrote: >>> I developed this patch while working on the dwarf2 pass series. >>> This was before I bypassed the entire problem by removing the >>> !

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-09 Thread Richard Henderson
On 07/09/2011 06:42 PM, H.J. Lu wrote: > On Sat, Jul 9, 2011 at 6:34 PM, Richard Henderson 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 generi

Re: [rfc, i386] Convert output_mi_thunk to rtl

2011-07-09 Thread H.J. Lu
On Sat, Jul 9, 2011 at 6:34 PM, Richard Henderson 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

[rfc, i386] Convert output_mi_thunk to rtl

2011-07-09 Thread Richard Henderson
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 wit