[llvm-commits] [llvm] r99655 - in /llvm/trunk/lib/Target/ARM: ARMInstrFormats.td ARMInstrNEON.td
Bob Wilson
bob.wilson at apple.com
Fri Mar 26 15:34:05 PDT 2010
On Mar 26, 2010, at 3:10 PM, Johnny Chen wrote:
> There are 18 invocations of the N3VInt_* multiclasses, which means 18 places to push the format arguments.
> Do you still think this is the way to go than invoking 8 N3VSh_QHSD for 3-register vector shift clusters?
>
> There will be 24 places where we need to pass in the format argument.
Yes (even if it is 26 places). I generally prefer to keep the instructions as succinct as possible, but not at the cost of duplicating all those classes and multiclasses.
>
> If you still think so, I will do the modification, since I'm the one who caused it in the first place. :-)
Great. Please do then. Thanks!
>
> On Mar 26, 2010, at 3:02 PM, Bob Wilson wrote:
>
>> I would rather push the format argument into all the N3VInt instructions. We already have too many classes and multiclasses for Neon, so I really don't want to create any more. Can you do that or do you want me to do it?
>>
>> On Mar 26, 2010, at 2:26 PM, Johnny Chen wrote:
>>
>>> Author: johnny
>>> Date: Fri Mar 26 16:26:28 2010
>>> New Revision: 99655
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=99655&view=rev
>>> Log:
>>> Add N3RegVShFrm to represent 3-Register Vector Shift Instructions, which do not
>>> follow the N3RegFrm's operand order of D:Vd N:Vn M:Vm. The operand order of
>>> N3RegVShFrm is D:Vd M:Vm N:Vn (notice that M:Vm is the first src operand).
>>>
>>> Add a parent class N3Vf which requires passing a Format argument and which the
>>> N3V class is modified to inherit from. N3V class represents the "normal"
>>> 3-Register NEON Instructions with N3RegFrm.
>>>
>>> Also add a multiclass N3VSh_QHSD to represent clusters of NEON 3-Register Shift
>>> Instructions and replace 8 invocations with it.
>>>
>>> Modified:
>>> llvm/trunk/lib/Target/ARM/ARMInstrFormats.td
>>> llvm/trunk/lib/Target/ARM/ARMInstrNEON.td
>>>
>>> Modified: llvm/trunk/lib/Target/ARM/ARMInstrFormats.td
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMInstrFormats.td?rev=99655&r1=99654&r2=99655&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/Target/ARM/ARMInstrFormats.td (original)
>>> +++ llvm/trunk/lib/Target/ARM/ARMInstrFormats.td Fri Mar 26 16:26:28 2010
>>> @@ -67,6 +67,7 @@
>>> def N2RegVShLFrm : Format<36>;
>>> def N2RegVShRFrm : Format<37>;
>>> def N3RegFrm : Format<38>;
>>> +def N3RegVShFrm : Format<39>;
>>>
>>> // Misc flags.
>>>
>>> @@ -1603,11 +1604,11 @@
>>> let Inst{4} = op4;
>>> }
>>>
>>> -// NEON 3 vector register format.
>>> -class N3V<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op6, bit op4,
>>> - dag oops, dag iops, InstrItinClass itin,
>>> - string opc, string dt, string asm, string cstr, list<dag> pattern>
>>> - : NDataI<oops, iops, N3RegFrm, itin, opc, dt, asm, cstr, pattern> {
>>> +// NEON 3 vector register template, which requires a Format argument.
>>> +class N3Vf<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op6,bit op4,
>>> + dag oops, dag iops, Format f, InstrItinClass itin,
>>> + string opc, string dt, string asm, string cstr, list<dag> pattern>
>>> + : NDataI<oops, iops, f, itin, opc, dt, asm, cstr, pattern> {
>>> let Inst{24} = op24;
>>> let Inst{23} = op23;
>>> let Inst{21-20} = op21_20;
>>> @@ -1616,6 +1617,13 @@
>>> let Inst{4} = op4;
>>> }
>>>
>>> +// NEON 3 vector register format.
>>> +class N3V<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op6, bit op4,
>>> + dag oops, dag iops, InstrItinClass itin,
>>> + string opc, string dt, string asm, string cstr, list<dag> pattern>
>>> + : N3Vf<op24, op23, op21_20, op11_8, op6, op4, oops, iops, N3RegFrm, itin,
>>> + opc, dt, asm, cstr, pattern>;
>>> +
>>> // Same as N3V except it doesn't have a data type suffix.
>>> class N3VX<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op6,
>>> bit op4,
>>>
>>> Modified: llvm/trunk/lib/Target/ARM/ARMInstrNEON.td
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMInstrNEON.td?rev=99655&r1=99654&r2=99655&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/Target/ARM/ARMInstrNEON.td (original)
>>> +++ llvm/trunk/lib/Target/ARM/ARMInstrNEON.td Fri Mar 26 16:26:28 2010
>>> @@ -1594,6 +1594,60 @@
>>> v2i64, v2i64, IntOp, Commutable>;
>>> }
>>>
>>> +// N3VSh_QHSD is similar to N3VInt_QHSD, except that it is for 3-Register Vector
>>> +// Shift Instructions (N3RegVShFrm), which do not follow the N3RegFrm's operand
>>> +// order of D:Vd N:Vn M:Vm.
>>> +//
>>> +// The operand order of N3RegVShFrm is D:Vd M:Vm N:Vn (notice that M:Vm is the
>>> +// first src operand).
>>> +class N3VDSh<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op4,
>>> + InstrItinClass itin, string OpcodeStr, string Dt,
>>> + ValueType ResTy, ValueType OpTy, Intrinsic IntOp, bit Commutable>
>>> + : N3Vf<op24, op23, op21_20, op11_8, 0, op4,
>>> + (outs DPR:$dst), (ins DPR:$src1, DPR:$src2), N3RegVShFrm,
>>> + itin, OpcodeStr, Dt, "$dst, $src1, $src2", "",
>>> + [(set DPR:$dst, (ResTy (IntOp (OpTy DPR:$src1), (OpTy DPR:$src2))))]> {
>>> + let isCommutable = Commutable;
>>> +}
>>> +class N3VQSh<bit op24, bit op23, bits<2> op21_20, bits<4> op11_8, bit op4,
>>> + InstrItinClass itin, string OpcodeStr, string Dt,
>>> + ValueType ResTy, ValueType OpTy, Intrinsic IntOp, bit Commutable>
>>> + : N3Vf<op24, op23, op21_20, op11_8, 1, op4,
>>> + (outs QPR:$dst), (ins QPR:$src1, QPR:$src2), N3RegVShFrm,
>>> + itin, OpcodeStr, Dt, "$dst, $src1, $src2", "",
>>> + [(set QPR:$dst, (ResTy (IntOp (OpTy QPR:$src1), (OpTy QPR:$src2))))]> {
>>> + let isCommutable = Commutable;
>>> +}
>>> +multiclass N3VSh_QHSD<bit op24, bit op23, bits<4> op11_8, bit op4,
>>> + InstrItinClass itinD16, InstrItinClass itinD32,
>>> + InstrItinClass itinQ16, InstrItinClass itinQ32,
>>> + string OpcodeStr, string Dt,
>>> + Intrinsic IntOp, bit Commutable> {
>>> + def v4i16 : N3VDSh<op24, op23, 0b01, op11_8, op4, itinD16,
>>> + OpcodeStr, !strconcat(Dt, "16"),
>>> + v4i16, v4i16, IntOp, Commutable>;
>>> + def v2i32 : N3VDSh<op24, op23, 0b10, op11_8, op4, itinD32,
>>> + OpcodeStr, !strconcat(Dt, "32"),
>>> + v2i32, v2i32, IntOp, Commutable>;
>>> + def v8i16 : N3VQSh<op24, op23, 0b01, op11_8, op4, itinQ16,
>>> + OpcodeStr, !strconcat(Dt, "16"),
>>> + v8i16, v8i16, IntOp, Commutable>;
>>> + def v4i32 : N3VQSh<op24, op23, 0b10, op11_8, op4, itinQ32,
>>> + OpcodeStr, !strconcat(Dt, "32"),
>>> + v4i32, v4i32, IntOp, Commutable>;
>>> + def v8i8 : N3VDSh<op24, op23, 0b00, op11_8, op4, itinD16,
>>> + OpcodeStr, !strconcat(Dt, "8"),
>>> + v8i8, v8i8, IntOp, Commutable>;
>>> + def v16i8 : N3VQSh<op24, op23, 0b00, op11_8, op4, itinQ16,
>>> + OpcodeStr, !strconcat(Dt, "8"),
>>> + v16i8, v16i8, IntOp, Commutable>;
>>> + def v1i64 : N3VDSh<op24, op23, 0b11, op11_8, op4,
>>> + itinD32, OpcodeStr, !strconcat(Dt, "64"),
>>> + v1i64, v1i64, IntOp, Commutable>;
>>> + def v2i64 : N3VQSh<op24, op23, 0b11, op11_8, op4,
>>> + itinQ32, OpcodeStr, !strconcat(Dt, "64"),
>>> + v2i64, v2i64, IntOp, Commutable>;
>>> +}
>>>
>>> // Neon Narrowing 3-register vector intrinsics,
>>> // source operand element sizes of 16, 32 and 64 bits:
>>> @@ -2575,10 +2629,10 @@
>>> // Vector Shifts.
>>>
>>> // VSHL : Vector Shift
>>> -defm VSHLs : N3VInt_QHSD<0, 0, 0b0100, 0, IIC_VSHLiD, IIC_VSHLiD, IIC_VSHLiQ,
>>> - IIC_VSHLiQ, "vshl", "s", int_arm_neon_vshifts, 0>;
>>> -defm VSHLu : N3VInt_QHSD<1, 0, 0b0100, 0, IIC_VSHLiD, IIC_VSHLiD, IIC_VSHLiQ,
>>> - IIC_VSHLiQ, "vshl", "u", int_arm_neon_vshiftu, 0>;
>>> +defm VSHLs : N3VSh_QHSD<0, 0, 0b0100, 0, IIC_VSHLiD, IIC_VSHLiD, IIC_VSHLiQ,
>>> + IIC_VSHLiQ, "vshl", "s", int_arm_neon_vshifts, 0>;
>>> +defm VSHLu : N3VSh_QHSD<1, 0, 0b0100, 0, IIC_VSHLiD, IIC_VSHLiD, IIC_VSHLiQ,
>>> + IIC_VSHLiQ, "vshl", "u", int_arm_neon_vshiftu, 0>;
>>> // VSHL : Vector Shift Left (Immediate)
>>> defm VSHLi : N2VSh_QHSD<0, 1, 0b0101, 1, IIC_VSHLiD, "vshl", "i", NEONvshl,
>>> N2RegVShLFrm>;
>>> @@ -2612,10 +2666,10 @@
>>> NEONvshrn>;
>>>
>>> // VRSHL : Vector Rounding Shift
>>> -defm VRSHLs : N3VInt_QHSD<0,0,0b0101,0, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vrshl", "s", int_arm_neon_vrshifts,0>;
>>> -defm VRSHLu : N3VInt_QHSD<1,0,0b0101,0, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vrshl", "u", int_arm_neon_vrshiftu,0>;
>>> +defm VRSHLs : N3VSh_QHSD<0,0,0b0101,0,IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q,"vrshl", "s", int_arm_neon_vrshifts,0>;
>>> +defm VRSHLu : N3VSh_QHSD<1,0,0b0101,0,IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q,"vrshl", "u", int_arm_neon_vrshiftu,0>;
>>> // VRSHR : Vector Rounding Shift Right
>>> defm VRSHRs : N2VSh_QHSD<0,1,0b0010,1, IIC_VSHLi4D, "vrshr", "s", NEONvrshrs,
>>> N2RegVShRFrm>;
>>> @@ -2627,10 +2681,10 @@
>>> NEONvrshrn>;
>>>
>>> // VQSHL : Vector Saturating Shift
>>> -defm VQSHLs : N3VInt_QHSD<0,0,0b0100,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vqshl", "s", int_arm_neon_vqshifts,0>;
>>> -defm VQSHLu : N3VInt_QHSD<1,0,0b0100,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vqshl", "u", int_arm_neon_vqshiftu,0>;
>>> +defm VQSHLs : N3VSh_QHSD<0,0,0b0100,1,IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q, "vqshl", "s", int_arm_neon_vqshifts,0>;
>>> +defm VQSHLu : N3VSh_QHSD<1,0,0b0100,1,IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q, "vqshl", "u", int_arm_neon_vqshiftu,0>;
>>> // VQSHL : Vector Saturating Shift Left (Immediate)
>>> defm VQSHLsi : N2VSh_QHSD<0,1,0b0111,1, IIC_VSHLi4D, "vqshl", "s",NEONvqshls,
>>> N2RegVShLFrm>;
>>> @@ -2651,12 +2705,12 @@
>>> NEONvqshrnsu>;
>>>
>>> // VQRSHL : Vector Saturating Rounding Shift
>>> -defm VQRSHLs : N3VInt_QHSD<0,0,0b0101,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vqrshl", "s",
>>> - int_arm_neon_vqrshifts, 0>;
>>> -defm VQRSHLu : N3VInt_QHSD<1,0,0b0101,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> - IIC_VSHLi4Q, "vqrshl", "u",
>>> - int_arm_neon_vqrshiftu, 0>;
>>> +defm VQRSHLs : N3VSh_QHSD<0,0,0b0101,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q, "vqrshl", "s",
>>> + int_arm_neon_vqrshifts, 0>;
>>> +defm VQRSHLu : N3VSh_QHSD<1,0,0b0101,1, IIC_VSHLi4D, IIC_VSHLi4D, IIC_VSHLi4Q,
>>> + IIC_VSHLi4Q, "vqrshl", "u",
>>> + int_arm_neon_vqrshiftu, 0>;
>>>
>>> // VQRSHRN : Vector Saturating Rounding Shift Right and Narrow
>>> defm VQRSHRNs : N2VNSh_HSD<0, 1, 0b1001, 0, 1, 1, IIC_VSHLi4D, "vqrshrn", "s",
>>>
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
More information about the llvm-commits
mailing list