[PATCH] [AArch64] Add support for TargetTransformInfo Analysis.

Chad Rosier mcrosier at codeaurora.org
Thu Feb 20 07:32:17 PST 2014


> LGTM with one question:
>
> +  unsigned getRegisterBitWidth(bool Vector) const {
> +    if (Vector) {
> +      if (ST->hasNEON())
> +        return 128;
> +      return 0;
> +    }
> +    return 32;
>
> Hmm, AArch64 has 32-bit width general purpose registers?

Yup, and 64.. ;)  Currently, this API is used exclusively by the loop
vectorizer to ask for the width of a vector register.  I'll return 64 in
the scalar case, but this has no functional change as far as I can tell. 
However, I do believe it would be the expected result.  Thanks, Chandler.

 Chad

>
>
> On Wed, Feb 19, 2014 at 11:57 AM, Chad Rosier
> <mcrosier at codeaurora.org>wrote:
>
>> All,
>> The attached patch adds the TargetTransformInfo analysis pass in the
>> AArch64 backend.  We hope to continue better defining the APIs as we
>> move
>> forward.
>>
>>  Regards,
>>   Chad
>>
>> --
>> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
>> hosted by The Linux Foundation
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>>
>


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation




More information about the llvm-commits mailing list