RE: [Patch 1/6] Hookize MOVE_BY_PIECES_P, remove most uses of MOVE_RATIO

2014-10-31 Thread James Greenhalgh
On Wed, Oct 29, 2014 at 03:31:54PM +, James Greenhalgh wrote: > On Wed, Oct 29, 2014 at 11:42:06AM +, Matthew Fortune wrote: > > Hi James, > > > > I think you have a bug in the following hunk where you pass > > STORE_MAX_PIECES in place of the optimise for speed flag. I guess you > > would

Re: [Patch 1/6] Hookize MOVE_BY_PIECES_P, remove most uses of MOVE_RATIO

2014-10-29 Thread James Greenhalgh
On Wed, Oct 29, 2014 at 11:42:06AM +, Matthew Fortune wrote: > Hi James, > > I think you have a bug in the following hunk where you pass > STORE_MAX_PIECES in place of the optimise for speed flag. I guess you > would need an extra argument to pass a different *_MAX_PIECES value > in. Yup, goo

RE: [Patch 1/6] Hookize MOVE_BY_PIECES_P, remove most uses of MOVE_RATIO

2014-10-29 Thread Matthew Fortune
Hi James, I think you have a bug in the following hunk where you pass STORE_MAX_PIECES in place of the optimise for speed flag. I guess you would need an extra argument to pass a different *_MAX_PIECES value in. thanks, Matthew >@@ -192,8 +184,7 @@ static void write_complex_part (rtx, rtx, bool)