[Bug middle-end/60080] New: gcc.dg/vect/vect-nop-move.c FAILs

[Bug middle-end/60080] New: gcc.dg/vect/vect-nop-move.c FAILs

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

            Bug ID: 60080
           Summary: gcc.dg/vect/vect-nop-move.c FAILs
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: ro at gcc dot gnu.org
              Host: i386-pc-solaris2.9, i386-pc-solaris2.10
            Target: i386-pc-solaris2.9, i386-pc-solaris2.10
             Build: i386-pc-solaris2.9, i386-pc-solaris2.10

The gcc.dg/vect/vect-nop-move.c FAILs on both Solaris 9 and 10/x86:

FAIL: gcc.dg/vect/vect-nop-move.c (internal compiler error)
FAIL: gcc.dg/vect/vect-nop-move.c (test for excess errors)
WARNING: gcc.dg/vect/vect-nop-move.c compilation failed to produce executable
FAIL: gcc.dg/vect/vect-nop-move.c -flto -ffat-lto-objects (internal compiler
error)
FAIL: gcc.dg/vect/vect-nop-move.c -flto -ffat-lto-objects (test for excess
errors)
WARNING: gcc.dg/vect/vect-nop-move.c -flto -ffat-lto-objects compilation failed
to produce executable

The ICE can be reproduced with

$ cc1 -fpreprocessed vect-nop-move.i -quiet -msse2 -O2
-fdump-rtl-combine-details -o vect-nop-move.s

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 1 (LWP 1)]
0xfee0646c in strlen () from /lib/libc.so.1
(gdb) where
#0  0xfee0646c in strlen () from /lib/libc.so.1
#1  0xfee61bce in _ndoprnt () from /lib/libc.so.1
#2  0xfee6477e in fprintf () from /lib/libc.so.1
#3  0x085641ea in print_rtx (in_rtx=0xfeaf718c)
    at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:430
#4  0x0856339c in print_rtx (in_rtx=0xfea8a800)
    at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:384
#5  0x0856331f in print_rtx (in_rtx=0xfeadfdd4)
    at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:363
#6  0x0856339c in print_rtx (in_rtx=in_rtx@entry=0xfea9bea8)
    at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:384
#7  0x08564814 in print_rtl_single_with_indent (outf=0x92e8100 <_iob+64>, 
    x=0xfea9bea8, ind=0) at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:836
#8  0x08564873 in print_rtl_single (outf=0x92e8100 <_iob+64>, x=0xfea9bea8)
    at /vol/gcc/src/hg/trunk/local/gcc/print-rtl.c:818
#9  0x08a7608b in recog_for_combine (pnewpat=pnewpat@entry=0x8047230, 
    insn=insn@entry=0xfeaf6264, pnotes=pnotes@entry=0x8047254)
    at /vol/gcc/src/hg/trunk/local/gcc/combine.c:10579
#10 0x08a879ea in try_combine (i3=i3@entry=0xfeaf6264, i2=<optimized out>, 
    i1=i1@entry=0x0, i0=<optimized out>, new_direct_jump_p=0x8047328, 
    last_combined_insn=0xfeaf6264)
    at /vol/gcc/src/hg/trunk/local/gcc/combine.c:3240
#11 0x08a8c2f4 in combine_instructions (nregs=<optimized out>, 
    f=<optimized out>) at /vol/gcc/src/hg/trunk/local/gcc/combine.c:1256
#12 rest_of_handle_combine ()
    at /vol/gcc/src/hg/trunk/local/gcc/combine.c:13865
#13 (anonymous namespace)::pass_combine::execute (this=0x9316930)
    at /vol/gcc/src/hg/trunk/local/gcc/combine.c:13908
#14 0x0854d8d1 in execute_one_pass (pass=0x9316930)
    at /vol/gcc/src/hg/trunk/local/gcc/passes.c:2229
#15 0x0854db6f in execute_pass_list (pass=<optimized out>)
    at /vol/gcc/src/hg/trunk/local/gcc/passes.c:2282
#16 0x0854db86 in execute_pass_list (pass=<optimized out>)
    at /vol/gcc/src/hg/trunk/local/gcc/passes.c:2283
#17 0x082df681 in expand_function (node=node@entry=0xfea08c7c)
    at /vol/gcc/src/hg/trunk/local/gcc/cgraphunit.c:1774
#18 0x082e1619 in expand_all_functions ()
    at /vol/gcc/src/hg/trunk/local/gcc/cgraphunit.c:1908
#19 compile () at /vol/gcc/src/hg/trunk/local/gcc/cgraphunit.c:2252
#20 0x082e1deb in finalize_compilation_unit ()
    at /vol/gcc/src/hg/trunk/local/gcc/cgraphunit.c:2329
#21 0x081b4aab in c_write_global_declarations ()
    at /vol/gcc/src/hg/trunk/local/gcc/c/c-decl.c:10401
#22 0x085f346d in compile_file ()
    at /vol/gcc/src/hg/trunk/local/gcc/toplev.c:562
#23 0x085f5526 in do_compile ()
    at /vol/gcc/src/hg/trunk/local/gcc/toplev.c:1914
#24 toplev_main (argc=9, argv=0x8047548)
    at /vol/gcc/src/hg/trunk/local/gcc/toplev.c:1990
#25 0x08d959ab in main (argc=9, argv=0x8047548)
    at /vol/gcc/src/hg/trunk/local/gcc/main.c:36

There's an attempt to print a NULL string.  On Solaris 11, the test works and
in the vect-nop-move.c.201r.combine dump there are many instances of (null),
e.g.

Trying 25 -> 26:
Failed to match this instruction:
(parallel [
        (set (reg:SI 97 [ __eax ])
            (asm_operands:SI ("cpuid
        ") ("=a") 0 [
                    (const_int 0 [0])
                ]
                 [
                    (asm_input:SI ("0") (null):0)
                ]
                 []
/var/gcc/regression/trunk/11-gcc/build/gcc/include/cpuid.h:248))

  Rainer

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.0

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[bernd.edlinger at hotmail dot de]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

Bernd Edlinger <bernd.edlinger at hotmail dot de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bernd.edlinger at hotmail dot de

--- Comment #1 from Bernd Edlinger <bernd.edlinger at hotmail dot de> ---
Richard,

how about this?

--- gcc/cfgexpand.c.jj    2014-01-09 21:12:36.000000000 +0100
+++ gcc/cfgexpand.c    2014-02-06 13:01:09.280392442 +0100
@@ -2678,8 +2678,9 @@ expand_asm_operands (tree string, tree o
       ASM_OPERANDS_INPUT (body, i) = op;

       ASM_OPERANDS_INPUT_CONSTRAINT_EXP (body, i)
-    = gen_rtx_ASM_INPUT (TYPE_MODE (type),
-                 ggc_strdup (constraints[i + noutputs]));
+    = gen_rtx_ASM_INPUT_loc (TYPE_MODE (type),
+                 ggc_strdup (constraints[i + noutputs]),
+                 locus);

       if (tree_conflicts_with_clobbers_p (val, &clobbered_regs))
     clobber_conflict_found = 1;
@@ -2701,7 +2702,7 @@ expand_asm_operands (tree string, tree o

       sprintf (buffer, "%d", j);
       ASM_OPERANDS_INPUT_CONSTRAINT_EXP (body, ninputs - ninout + i)
-    = gen_rtx_ASM_INPUT (inout_mode[i], ggc_strdup (buffer));
+    = gen_rtx_ASM_INPUT_loc (inout_mode[i], ggc_strdup (buffer), locus);
     }

   /* Copy labels to the vector.  */

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[ro at CeBiTec dot Uni-Bielefeld.DE]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

--- Comment #2 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
I just tried the patch on i386-pc-solaris2.10 and the SEGVs are gone.

Thanks for the quick fix.

    Rainer

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[edlinger at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

--- Comment #3 from edlinger at gcc dot gnu.org ---
Author: edlinger
Date: Mon Feb 10 18:19:01 2014
New Revision: 207668

URL: http://gcc.gnu.org/viewcvs?rev=207668&root=gcc&view=rev
Log:
2014-02-10  Bernd Edlinger  <bernd.edlinger@hotmail.de>

        PR middle-end/60080
        * cfgexpand.c (expand_asm_operands): Attach source location to
        ASM_INPUT rtx objects.
        * print-rtl.c (print_rtx): Check for UNKNOWN_LOCATION.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cfgexpand.c
    trunk/gcc/print-rtl.c

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.9.0                       |4.9.1

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.0 has been released

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[bernd.edlinger at hotmail dot de]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

--- Comment #5 from Bernd Edlinger <bernd.edlinger at hotmail dot de> ---
aehm...
actually this issue is already fixed,
but I can't set the status to FIXED.

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.9.1                       |4.9.2

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.1 has been released.

[Bug middle-end/60080] gcc.dg/vect/vect-nop-move.c FAILs

[edlinger at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60080

edlinger at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #7 from edlinger at gcc dot gnu.org ---
has been fixed in 4.9.0