Hi Richard, I was wondering if you got a chance to see if this new patch is alright ?.
Thanks, -Sriraman. On Thu, Oct 1, 2009 at 2:37 PM, Sriraman Tallam <tmsri...@google.com> wrote: > Hi, > > I moved implicit-zee.c to config/i386. Can you please take another look ? > > * tree-pass.h (pass_implicit_zee): New pass. > * testsuite/gcc.target/i386/zee.c: New test. > * timevar.def (TV_ZEE): New. > * common.opt (fzee): New flag. > * config.gcc: Add implicit-zee.o for x86_64 target. > * implicit-zee.c: New file, zero extension elimination pass. > * config/i386/t-i386: Add rule for implicit-zee.o. > * i386.c (optimization_options): Enable zee pass for x86_64 target. > > Thanks, > -Sriraman. > > > On Thu, Sep 24, 2009 at 9:34 AM, Sriraman Tallam <tmsri...@google.com> wrote: >> On Thu, Sep 24, 2009 at 1:36 AM, Richard Guenther >> <richard.guent...@gmail.com> wrote: >>> On Thu, Sep 24, 2009 at 8:25 AM, Paolo Bonzini <bonz...@gnu.org> wrote: >>>> On 09/24/2009 08:24 AM, Ian Lance Taylor wrote: >>>>> >>>>> We already have the hooks, they have just been stuck in plugin.c when >>>>> they should really be in the generic backend. See register_pass. >>>>> >>>>> (Sigh, every time I looked at this I said "the pass control has to be >>>>> generic" but it still wound up in plugin.c.) >>>> >>>> Then I'll rephrase and say only that the pass should be in config/i386/. >>> >>> It should also be on by default on -O[23s] I think (didn't check if it >>> already >>> is). Otherwise it shortly will go the see lala-land. >> >> It is already on by default in O2 and higher. >> >>> >>> Richard. >>> >>>> Paolo >>>> >>> >> >