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 <[email protected]> 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 <[email protected]> wrote:
>> On Thu, Sep 24, 2009 at 1:36 AM, Richard Guenther
>> <[email protected]> wrote:
>>> On Thu, Sep 24, 2009 at 8:25 AM, Paolo Bonzini <[email protected]> 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
>>>>
>>>
>>
>
