[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