This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[Obvious ARM Testsuite] mangle-arm-crypto.C needs arm_crypto arguments added
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: ramana dot radhakrishnan at arm dot com, richard dot earnshaw at arm dot com
- Date: Fri, 16 Jan 2015 17:38:25 +0000
- Subject: [Obvious ARM Testsuite] mangle-arm-crypto.C needs arm_crypto arguments added
- Authentication-results: sourceware.org; auth=none
Hi,
g++.dg/abi/mangle-arm-crypto.C fails on some ARM targets
(see: https://gcc.gnu.org/ml/gcc-testresults/2015-01/msg01654.html ).
Much to my irritation, I can't reproduce the failures locally, but the
reason looks simple enough - you need to pass the correct flags to
enable the cypto intrinsics and instructions.
There is already a dg-add-options directive for this, so use it.
I've committed this patch as obvious as r219758, after checking it doesn't
cause any new problems on the platforms I have access to. I'll watch
Ramana's test results over the weekend to see if this fixes his bug!
Cheers,
James
---
gcc/testsuite/
2015-01-16 James Greenhalgh <james.greenhalgh@arm.com>
* g++.dg/abi/mangle-abi-crypto.C: Add crypto options, rather
than Neon options.
diff --git a/gcc/testsuite/g++.dg/abi/mangle-arm-crypto.C b/gcc/testsuite/g++.dg/abi/mangle-arm-crypto.C
index aae8847..f3fb1a9 100644
--- a/gcc/testsuite/g++.dg/abi/mangle-arm-crypto.C
+++ b/gcc/testsuite/g++.dg/abi/mangle-arm-crypto.C
@@ -3,7 +3,7 @@
// { dg-do compile }
// { dg-require-effective-target arm_crypto_ok }
-// { dg-add-options arm_neon }
+// { dg-add-options arm_crypto }
#include <arm_neon.h>