[Bug target/53425] New: No warnings are given for -mno-sse

[Bug target/53425] New: No warnings are given for -mno-sse

[hjl.tools at gmail dot com]

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.

[Bug target/53425] No warnings are given for -mno-sse

[hjl at gcc dot gnu.org]

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

[Bug target/53425] No warnings are given for -mno-sse

[hjl.tools at gmail dot com]

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.

[Bug target/53425] No warnings are given for -mno-sse

[jon_y at users dot sourceforge.net]

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?