[match-and-simplify] CSE with expression captures

Prathamesh Kulkarni bilbotheelffriend@gmail.com
Tue Sep 16 06:25:00 GMT 2014


On Tue, Sep 16, 2014 at 2:15 AM, Marc Glisse <marc.glisse@inria.fr> wrote:
> On Tue, 16 Sep 2014, Prathamesh Kulkarni wrote:
>
>> --- gcc/match-builtin.pd        (revision 215271)
>> +++ gcc/match-builtin.pd        (working copy)
>> @@ -44,8 +44,8 @@
>>           /* ???  There is no way to CSE here.  We'd need to support
>>              expression captures here, like with
>>               (mult (realpart@1 @0) @1) */
>> -         (mult (realpart @0) (realpart @0))
>> -         (mult (imagpart @0) (imagpart @0)))))))
>> +         (mult (realpart@1 @0) @1) +     (mult (imagpart@2 @0) @2))))))
>
>
> Maybe remove the comment above?
>
> You seem to have trailing spaces on most of your new lines.
Thanks, fixed.

* genmatch.c (operand::gen_transform): Add dt_operand ** default argument
to operand heirarchy.
  (expr::gen_transform): Adjust.
  (capture::gen_transform): Likewise.
  (dt_simplify::gen): Likewise.

* match-builtins.pd: Adjust pattern to use expression captures in
transform.

Regards,
Prathamesh
>
> --
> Marc Glisse
-------------- next part --------------
A non-text attachment was scrubbed...
Name: expr-capture2.patch
Type: text/x-patch
Size: 5046 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140916/099e630c/attachment.bin>


More information about the Gcc-patches mailing list