On 10/09/14 09:16, Bin.Cheng wrote:
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.
No, that can't be; these definitions are in arm.c, so can't impact aarch64.
R.
Thanks,
bin
