[patch 2/2] AMD bdver2 processors - TBM

Richard Henderson rth@redhat.com
Thu Nov 4 18:07:00 GMT 2010


On 11/04/2010 09:43 AM, Quentin Neill wrote:
>> Watch the extra vertical whitespace.
> 
> So should I have two lines (to separate list of ';TBM instructions')
> or just one?

Just one.

> So pulling the two bytes out in the builtin expander - should that be
> done with a *new* define_expand?  Or is there a set of RTX operations
> (shift, mask) I should use in the define_insn as written?

There's a set of C operators you may use.  ;-)

This is an *immediate*.  It's value is INTVAL (op1).
You can pull bits out from that and use GEN_INT to
pass the new rtl on to the define_insn.


r~



More information about the Gcc-patches mailing list