public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug target/53425] New: No warnings are given for -mno-sse @ 2012-05-20 14:18 hjl.tools at gmail dot com 2012-05-21 13:00 ` [Bug target/53425] " hjl at gcc dot gnu.org ` (2 more replies) 0 siblings, 3 replies; 4+ messages in thread From: hjl.tools at gmail dot com @ 2012-05-20 14:18 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53425 Bug #: 53425 Summary: No warnings are given for -mno-sse Classification: Unclassified Product: gcc Version: 4.8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target AssignedTo: unassigned@gcc.gnu.org ReportedBy: hjl.tools@gmail.com CC: ubizjak@gmail.com Target: x86-64 [hjl@gnu-mic-2 pr53383]$ cat z.c typedef double __v2df __attribute__ ((__vector_size__ (16))); extern __v2df x; extern void bar (__v2df); void foo (void) { bar (x); } [hjl@gnu-mic-2 pr53383]$ make z.s /export/build/gnu/gcc/build-x86_64-linux/gcc/xgcc -B/export/build/gnu/gcc/build-x86_64-linux/gcc/ -O2 -mno-sse -S z.c [hjl@gnu-mic-2 pr53383]$ cat z.s .file "z.c" .text .p2align 4,,15 .globl foo .type foo, @function foo: .LFB0: .cfi_startproc subq $40, %rsp .cfi_def_cfa_offset 48 movq x(%rip), %rax movq %rax, (%rsp) movq x+8(%rip), %rax movq %rax, 8(%rsp) call bar addq $40, %rsp .cfi_def_cfa_offset 8 ret .cfi_endproc .LFE0: .size foo, .-foo .ident "GCC: (GNU) 4.8.0 20120519 (experimental)" .section .note.GNU-stack,"",@progbits [hjl@gnu-mic-2 pr53383]$ But we do issue a warning for 32-bit: [hjl@gnu-mic-2 pr53383]$ /export/build/gnu/gcc/build-x86_64-linux/gcc/xgcc -B/export/build/gnu/gcc/build-x86_64-linux/gcc/ -O2 -mno-sse -S z.c -m32 z.c: In function \u2018foo\u2019: z.c:9:7: warning: SSE vector argument without SSE enabled changes the ABI [enabled by default] bar (x); ^ z.c:9:7: note: The ABI for passing parameters with 16-byte alignment has changed in GCC 4.6 [hjl@gnu-mic-2 pr53383]$ I believe we should also issue a warning for 64-bit. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/53425] No warnings are given for -mno-sse 2012-05-20 14:18 [Bug target/53425] New: No warnings are given for -mno-sse hjl.tools at gmail dot com @ 2012-05-21 13:00 ` hjl at gcc dot gnu.org 2012-05-21 13:14 ` hjl.tools at gmail dot com 2012-06-14 12:37 ` jon_y at users dot sourceforge.net 2 siblings, 0 replies; 4+ messages in thread From: hjl at gcc dot gnu.org @ 2012-05-21 13:00 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53425 --- Comment #1 from hjl at gcc dot gnu.org <hjl at gcc dot gnu.org> 2012-05-21 12:48:43 UTC --- Author: hjl Date: Mon May 21 12:48:38 2012 New Revision: 187711 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187711 Log: Warn passing SSE argument ABI change for -m64 gcc/ PR target/53425 * config/i386/i386.c (type_natural_mode): Warn passing SSE vector argument without SSE enabled changes the ABI. gcc/testsuite/ PR target/53425 * gcc.target/i386/pr53425-1.c: New file. * gcc.target/i386/pr53425-2.c: Likewise. Added: trunk/gcc/testsuite/gcc.target/i386/pr53425-1.c trunk/gcc/testsuite/gcc.target/i386/pr53425-2.c Modified: trunk/gcc/ChangeLog trunk/gcc/config/i386/i386.c trunk/gcc/testsuite/ChangeLog ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/53425] No warnings are given for -mno-sse 2012-05-20 14:18 [Bug target/53425] New: No warnings are given for -mno-sse hjl.tools at gmail dot com 2012-05-21 13:00 ` [Bug target/53425] " hjl at gcc dot gnu.org @ 2012-05-21 13:14 ` hjl.tools at gmail dot com 2012-06-14 12:37 ` jon_y at users dot sourceforge.net 2 siblings, 0 replies; 4+ messages in thread From: hjl.tools at gmail dot com @ 2012-05-21 13:14 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53425 H.J. Lu <hjl.tools at gmail dot com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED Target Milestone|--- |4.8.0 --- Comment #2 from H.J. Lu <hjl.tools at gmail dot com> 2012-05-21 13:09:54 UTC --- Fixed. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/53425] No warnings are given for -mno-sse 2012-05-20 14:18 [Bug target/53425] New: No warnings are given for -mno-sse hjl.tools at gmail dot com 2012-05-21 13:00 ` [Bug target/53425] " hjl at gcc dot gnu.org 2012-05-21 13:14 ` hjl.tools at gmail dot com @ 2012-06-14 12:37 ` jon_y at users dot sourceforge.net 2 siblings, 0 replies; 4+ messages in thread From: jon_y at users dot sourceforge.net @ 2012-06-14 12:37 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53425 jon_y <jon_y at users dot sourceforge.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ktietz70 at googlemail dot | |com --- Comment #3 from jon_y <jon_y at users dot sourceforge.net> 2012-06-14 12:37:03 UTC --- Hi, I've been notified that this test fails on mingw-w64. Unfortunately, I am not familiar enough on Win64 ABI to comment on this. Kai, any ideas? ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-06-14 12:37 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-05-20 14:18 [Bug target/53425] New: No warnings are given for -mno-sse hjl.tools at gmail dot com 2012-05-21 13:00 ` [Bug target/53425] " hjl at gcc dot gnu.org 2012-05-21 13:14 ` hjl.tools at gmail dot com 2012-06-14 12:37 ` jon_y at users dot sourceforge.net
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).