On Tue, Sep 9, 2014 at 6:49 PM, Richard Earnshaw <rearn...@arm.com> wrote:
> On 04/09/14 07:08, Bin Cheng wrote:
>> @@ -1872,7 +1892,9 @@ const struct tune_params arm_cortex_a53_tune =
>> {true, true}, /* Prefer non short
>> circuit. */
>> &arm_default_vec_cost, /* Vectorizer costs. */
>> false, /* Prefer Neon for 64-bits
>> bitops. */
>> - false, false /* Prefer 32-bit encodings.
>> */
>> + false, false, /* Prefer 32-bit encodings.
>> */
>> + false, /* Prefer Neon for stringops.
>> */
>> + 8 /* Maximum insns to inline
>> memset. */
>> };
>>
>> const struct tune_params arm_cortex_a57_tune =
>> @@ -1889,7 +1911,9 @@ const struct tune_params arm_cortex_a57_tune =
>> {true, true}, /* Prefer non short circuit.
>> */
>> &arm_default_vec_cost, /* Vectorizer costs. */
>> false, /* Prefer Neon for 64-bits
>> bitops. */
>> - true, true /* Prefer 32-bit encodings.
>> */
>> + true, true, /* Prefer 32-bit encodings.
>> */
>> + false, /* Prefer Neon for stringops.
>> */
>> + 8 /* Maximum insns to inline
>> memset. */
>> };
>
> Why don't we use Neon for Cortex-(A53,A57)? We've used it for
> Cortex-(A7,A8,A15).
Hi Richard,
Since changes in aarch64.md is needed, also the bug and patch are for
arm only, I left this behind for aarch64. Another aarch64 patch
should be worked for this in the near (?) future.
Thanks,
bin
