Re: [patch] Split parts of cse_insn out to a few new functions

2012-03-26 Thread Steven Bosscher
On Mon, Mar 26, 2012 at 9:02 PM, Richard Sandiford wrote: >>       * cse.c (cse_canonicalized_basic_blocks): New simple bitmap to >>       tag basic blocks that have already been traversed at least once, >>       so that all insns have been canonicalized. >>       (cse_insn): Call canonicalize_ins

Re: [patch] Split parts of cse_insn out to a few new functions

2012-03-26 Thread Richard Sandiford
Steven Bosscher writes: > On Wed, Mar 21, 2012 at 1:13 AM, Ian Lance Taylor wrote: >> On Tue, Mar 20, 2012 at 2:06 PM, Steven Bosscher wrote: >>> >>> This patch splits a couple of pieces of cse_insn out to new functions. >>> There are no functional changes, and no code generation differences as >>

Re: [patch] Split parts of cse_insn out to a few new functions

2012-03-21 Thread Steven Bosscher
On Thu, Mar 22, 2012 at 12:09 AM, Steven Bosscher wrote: >        (cse_find_path): Micro-optimization, reorder one condition to >        avoid a reference to cfun. Ah, and please ignore this bit. I don't know what I was thinking...

Re: [patch] Split parts of cse_insn out to a few new functions

2012-03-21 Thread Steven Bosscher
On Wed, Mar 21, 2012 at 1:13 AM, Ian Lance Taylor wrote: > On Tue, Mar 20, 2012 at 2:06 PM, Steven Bosscher wrote: >> >> This patch splits a couple of pieces of cse_insn out to new functions. >> There are no functional changes, and no code generation differences as >> far as I could tell on x86_64

Re: [patch] Split parts of cse_insn out to a few new functions

2012-03-20 Thread Ian Lance Taylor
On Tue, Mar 20, 2012 at 2:06 PM, Steven Bosscher wrote: > > This patch splits a couple of pieces of cse_insn out to new functions. > There are no functional changes, and no code generation differences as > far as I could tell on x86_64 (-m64 and -m32). > > The purpose of the patch is and, loto hop