On Thu, Nov 14, 2019 at 05:44:42PM -0500, Michael Meissner wrote:
> This patch generates PADDI to add 34-bit immediate constants on the 'future'
> system, and prevents such adds from being split.

I don't see that last part?  Is that a remnant of a previous version of
the patch?

>       * config/rs6000/predicates.md (add_operand): Add support for
>       PADDI.

More directly: Allow constants that satisfy eI.

>       * config/rs6000/rs6000.md (add<mode>3): Add support for PADDI.

(add<mode>3 for GPR): etc.

We have six things all called add<mode>3 (VI2, DDTD, SDI, SFDF, IEEE128,
VEC_F).  Erm.

Wait, this is called *add<mode>3, not add<mode>3.


Okay for trunk with that fixed.  Thanks,


Segher

Reply via email to