[llvm] r251500 - [X86] Make some for loops over MVTs more explicit (and shorter) by just mentioning all the relevant types in an initializer list. NFC

[Craig Topper via llvm-commits]

Given that there are hundreds of other lines of code that explicitly
hardcode MVTs into calls to setOperationAction this seemed no worse to make
these sequences explicitly mentioned near the calls that are in loops.
These list aren't going to change as they are vector types supported in
SSE, AVX, and AVX2 which are defined by hardware.

Same with the integer ones in the other commit. We aren't suddenly going to
be able to support a different size operation.

On Wed, Oct 28, 2015 at 9:35 AM, David Blaikie <dblaikie at gmail.com> wrote:

>
>
> On Tue, Oct 27, 2015 at 10:48 PM, Craig Topper via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> Author: ctopper
>> Date: Wed Oct 28 00:48:32 2015
>> New Revision: 251500
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=251500&view=rev
>> Log:
>> [X86] Make some for loops over MVTs more explicit (and shorter) by just
>> mentioning all the relevant types in an initializer list. NFC
>>
>
> More duplication of this sequence - I'm still not sure that's an
> improvement? (but perhaps there's existing precedent that shows this is the
> right way)
>
>
>>
>> Modified:
>>     llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
>>
>> Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=251500&r1=251499&r2=251500&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
>> +++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Wed Oct 28 00:48:32 2015
>> @@ -858,14 +858,7 @@ X86TargetLowering::X86TargetLowering(con
>>      // ISD::CTTZ_ZERO_UNDEF v2i64 - scalarization is faster.
>>
>>      // Custom lower build_vector, vector_shuffle, and extract_vector_elt.
>> -    for (int i = MVT::v16i8; i != MVT::v2i64; ++i) {
>> -      MVT VT = (MVT::SimpleValueType)i;
>> -      // Do not attempt to custom lower non-power-of-2 vectors
>> -      if (!isPowerOf2_32(VT.getVectorNumElements()))
>> -        continue;
>> -      // Do not attempt to custom lower non-128-bit vectors
>> -      if (!VT.is128BitVector())
>> -        continue;
>> +    for (auto VT : { MVT::v16i8, MVT::v8i16, MVT::v4i32 }) {
>>        setOperationAction(ISD::BUILD_VECTOR,       VT, Custom);
>>        setOperationAction(ISD::VECTOR_SHUFFLE,     VT, Custom);
>>        setOperationAction(ISD::VSELECT,            VT, Custom);
>> @@ -903,13 +896,7 @@ X86TargetLowering::X86TargetLowering(con
>>      }
>>
>>      // Promote v16i8, v8i16, v4i32 load, select, and, or, xor to v2i64.
>> -    for (int i = MVT::v16i8; i != MVT::v2i64; ++i) {
>> -      MVT VT = (MVT::SimpleValueType)i;
>> -
>> -      // Do not attempt to promote non-128-bit vectors
>> -      if (!VT.is128BitVector())
>> -        continue;
>> -
>> +    for (auto VT : { MVT::v16i8, MVT::v8i16, MVT::v4i32 }) {
>>        setOperationAction(ISD::AND,    VT, Promote);
>>        AddPromotedToType (ISD::AND,    VT, MVT::v2i64);
>>        setOperationAction(ISD::OR,     VT, Promote);
>> @@ -1288,13 +1275,7 @@ X86TargetLowering::X86TargetLowering(con
>>        setOperationAction(ISD::VSELECT,         MVT::v32i8, Legal);
>>
>>      // Promote v32i8, v16i16, v8i32 select, and, or, xor to v4i64.
>> -    for (int i = MVT::v32i8; i != MVT::v4i64; ++i) {
>> -      MVT VT = (MVT::SimpleValueType)i;
>> -
>> -      // Do not attempt to promote non-256-bit vectors
>> -      if (!VT.is256BitVector())
>> -        continue;
>> -
>> +    for (auto VT : { MVT::v32i8, MVT::v16i16, MVT::v8i32 }) {
>>        setOperationAction(ISD::AND,    VT, Promote);
>>        AddPromotedToType (ISD::AND,    VT, MVT::v4i64);
>>        setOperationAction(ISD::OR,     VT, Promote);
>> @@ -1602,13 +1583,7 @@ X86TargetLowering::X86TargetLowering(con
>>          setOperationAction(ISD::MSTORE,              VT, Legal);
>>        }
>>      }
>> -    for (int i = MVT::v32i8; i != MVT::v8i64; ++i) {
>> -      MVT VT = (MVT::SimpleValueType)i;
>> -
>> -      // Do not attempt to promote non-512-bit vectors.
>> -      if (!VT.is512BitVector())
>> -        continue;
>> -
>> +    for (auto VT : { MVT::v64i8, MVT::v32i16, MVT::v16i32 }) {
>>        setOperationAction(ISD::SELECT, VT, Promote);
>>        AddPromotedToType (ISD::SELECT, VT, MVT::v8i64);
>>      }
>> @@ -1685,19 +1660,9 @@ X86TargetLowering::X86TargetLowering(con
>>        setOperationAction(ISD::CTLZ_ZERO_UNDEF, MVT::v64i8,  Custom);
>>      }
>>
>> -    for (int i = MVT::v32i8; i != MVT::v8i64; ++i) {
>> -      const MVT VT = (MVT::SimpleValueType)i;
>> -
>> -      const unsigned EltSize = VT.getVectorElementType().getSizeInBits();
>> -
>> -      // Do not attempt to promote non-512-bit vectors.
>> -      if (!VT.is512BitVector())
>> -        continue;
>> -
>> -      if (EltSize < 32) {
>> -        setOperationAction(ISD::BUILD_VECTOR,        VT, Custom);
>> -        setOperationAction(ISD::VSELECT,             VT, Legal);
>> -      }
>> +    for (auto VT : { MVT::v64i8, MVT::v32i16 }) {
>> +      setOperationAction(ISD::BUILD_VECTOR,        VT, Custom);
>> +      setOperationAction(ISD::VSELECT,             VT, Legal);
>>      }
>>    }
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>
>
>


-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151028/4d558f4f/attachment.html>