On Tue, Jun 12, 2012 at 8:45 AM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Mon, Jun 11, 2012 at 1:37 PM, William J. Schmidt > <wschm...@linux.vnet.ibm.com> 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 of a >> new one (with updated commentary). >> >> Bootstraps and tests with no new regressions on >> powerpc64-unknown-linux-gnu. One last performance run is underway, but >> I don't expect any surprises since both changes are more conservative. >> The original benchmark issue is still resolved. >> >> Is this version ok for trunk? >> >> Thanks, >> Bill >> >> >> 2012-06-11 Bill Schmidt <wschm...@linux.vnet.ibm.com> >> >> * opts.c: Add -fhoist-adjacent-loads to -O2 and above. >> * tree-ssa-phiopt.c (tree_ssa_phiopt_worker): Add argument to forward >> declaration. >> (hoist_adjacent_loads, gate_hoist_loads): New forward declarations. >> (tree_ssa_phiopt): Call gate_hoist_loads. >> (tree_ssa_cs_elim): Add parm to tree_ssa_phiopt_worker call. >> (tree_ssa_phiopt_worker): Add do_hoist_loads to formal arg list; call >> hoist_adjacent_loads. >> (local_mem_dependence): New function. >> (hoist_adjacent_loads): Likewise. >> (gate_hoist_loads): Likewise. >> * common.opt (fhoist-adjacent-loads): New switch. >> * Makefile.in (tree-ssa-phiopt.o): Added dependencies. >> >> > > This may have caused: > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53647 >
This also caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53703 -- H.J.
