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
>>>>
>>>
>>
>

Reply via email to