Hi Naveen,
It's a good idea to CC the AArch64 maintainers and reviewers on aarch64
patches, or at least
mention [AArch64] in the subject. I've cc'ed them for you.
On 08/11/16 05:47, Hurugalawadi, Naveen wrote:
Hi,
Please find attached the patch that fixes PR71727.
Please review the patch and let me know if its okay?
Some more information about why the current behaviour is wrong
and how the patch fixes it would be useful in reviewing.
Regression tested on Aarch64 with no regressions.
Thanks,
Naveen
2016-11-08 Naveen H.S <naveen.hurugalaw...@caviumnetworks.com>
* config/aarch64/aarch64.c
(aarch64_builtin_support_vector_misalignment): New.
(TARGET_VECTORIZE_SUPPORT_VECTOR_MISALIGNMENT): Define.
* gcc.target/aarch64/pr71727.c : New Testcase.
@@ -11148,6 +11152,35 @@ aarch64_simd_vector_alignment_reachable (const_tree
type, bool is_packed)
return true;
}
+static bool
+aarch64_builtin_support_vector_misalignment (machine_mode mode,
+ const_tree type, int misalignment,
+ bool is_packed)
+{
New functions need a function comment describing their arguments and their
result.
+ if (TARGET_SIMD && STRICT_ALIGNMENT)
+ {
+ /* Return if movmisalign pattern is not supported for this mode. */
+ if (optab_handler (movmisalign_optab, mode) == CODE_FOR_nothing)
+ return false;
+
Thanks,
Kyrill
