Re: [PATCH] Hoist adjacent loads

2012-06-17 Thread H.J. Lu
On Tue, Jun 12, 2012 at 8:45 AM, H.J. Lu wrote: > On Mon, Jun 11, 2012 at 1:37 PM, William J. Schmidt > wrote: >> OK, once more with feeling... :) >> >> This patch differs from the previous one in two respects:  It disables >> the optimization when either the then or else edge is well-predicted;

Re: [PATCH] Hoist adjacent loads

2012-06-12 Thread H.J. Lu
On Mon, Jun 11, 2012 at 1:37 PM, William J. Schmidt wrote: > OK, once more with feeling... :) > > This patch differs from the previous one in two respects:  It disables > the optimization when either the then or else edge is well-predicted; > and it now uses the existing l1-cache-line-size paramet

Re: [PATCH] Hoist adjacent loads

2012-06-12 Thread Richard Guenther
On Mon, 11 Jun 2012, William J. Schmidt wrote: > OK, once more with feeling... :) > > This patch differs from the previous one in two respects: It disables > the optimization when either the then or else edge is well-predicted; > and it now uses the existing l1-cache-line-size parameter instead

Re: [PATCH] Hoist adjacent loads

2012-06-11 Thread William J. Schmidt
OK, once more with feeling... :) This patch differs from the previous one in two respects: It disables the optimization when either the then or else edge is well-predicted; and it now uses the existing l1-cache-line-size parameter instead of a new one (with updated commentary). Bootstraps and te