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