https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #8 from Jakub Jelinek ---
Author: jakub
Date: Fri Jan 18 09:15:36 2019
New Revision: 268064
URL: https://gcc.gnu.org/viewcvs?rev=268064=gcc=rev
Log:
PR target/88734
* config/arm/arm_neon.h: Fix #pragma GCC target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #7 from Jakub Jelinek ---
Author: jakub
Date: Thu Jan 17 23:34:42 2019
New Revision: 268049
URL: https://gcc.gnu.org/viewcvs?rev=268049=gcc=rev
Log:
PR target/88734
* config/aarch64/arm_neon.h: Fix #pragma GCC target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #6 from Tamar Christina ---
Yes, I noticed them as well as was taking a look.
I don't quite understand what was going on before and why they passed but the
option is wrong. Those intrinsics and are all sha3 extensions.
diff --git
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #5 from Jakub Jelinek ---
+FAIL: gcc.target/aarch64/sha2_1.c (test for excess errors)
+UNRESOLVED: gcc.target/aarch64/sha2_1.c scan-assembler-times
sha512htq[0-9]+, q[0-9]+, v[0-9]+.2d 1
+UNRESOLVED: gcc.target/aarch64/sha2_1.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #4 from Tamar Christina ---
Hi Jakub,
Thanks for the patch, I've started a testrun but the fix looks sensible to me.
I'll post the result as soon as it's finished.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
--- Comment #3 from Jakub Jelinek ---
Actually, with -Wsystem-headers we do warn about that:
include/arm_neon.h:33073:9: warning: ‘#pragma GCC option’ is not a string
[-Wpragmas]
33073 | #pragma GCC target(("arch=armv8.2-a+sm4"))
|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88734
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P2