[Bug target/45212] New: FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[Bug target/45212] New: FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[ubizjak at gmail dot com]

gcc.target/alpha/PR24178.c started to fail on mainline recently:

FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

The problem is, that we don't generate expected ldl insn, but:

f:
        .frame $30,0,$26,0
        .prologue 0
        lda $1,18($16)
        ldq_u $0,18($16)
        extbl $0,$1,$0
        ret $31,($26),1
        .end f

The generated code is similar to what PR24178 fixed, see PR24178, comment #9.

The testcase passed with r161055 [1] and failed with r161806 [2].

In between is Richi's mem-ref2 merge...

[1] http://gcc.gnu.org/ml/gcc-testresults/2010-06/msg02114.html
[2] http://gcc.gnu.org/ml/gcc-testresults/2010-07/msg00417.html


-- 
           Summary: FAIL: gcc.target/alpha/pr24178.c scan-assembler
                    ldl.*,18\\\\(
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: ubizjak at gmail dot com
GCC target triplet: alphaev68-pc-linux-gnu


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

[Bug target/45212] [4.6 Regression] FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[ubizjak at gmail dot com]

------- Comment #1 from ubizjak at gmail dot com  2010-08-06 20:14 -------
The problem is, that we enter alpha_expand_mov_nobwx with operand[1]:

(mem/s:QI (plus:DI (reg/v/f:DI 72 [ p10 ])
        (const_int 18 [0x12])) [0+8 S1 A16])

This fails "aligned_memory_operand (operands[1], mode)" check.

OTOH, for gcc-4_5 branch, alpha_expand_mov_nobwx is called with:

(mem/s:QI (plus:DI (reg/v/f:DI 74 [ p10 ])
        (const_int 18 [0x12])) [0 S1 A64])


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-08-06 20:14:13
               date|                            |
            Summary|FAIL:                       |[4.6 Regression] FAIL:
                   |gcc.target/alpha/pr24178.c  |gcc.target/alpha/pr24178.c
                   |scan-assembler ldl.*,18\\\\(|scan-assembler ldl.*,18\\\\(
   Target Milestone|---                         |4.6.0


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

[Bug target/45212] [4.6 Regression] FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-08-06 20:29 -------
Mine.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2010-08-06 20:14:13         |2010-08-06 20:29:25
               date|                            |


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

[Bug target/45212] [4.6 Regression] FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2010-08-09 11:43 -------
Subject: Bug 45212

Author: rguenth
Date: Mon Aug  9 11:43:23 2010
New Revision: 163029

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163029
Log:
2010-08-09  Richard Guenther  <rguenther@suse.de>

        PR middle-end/45212
        * emit-rtl.c (set_mem_attributes_minus_bitpos): Adjust
        alignment from MEM_REF offset only if we took it from the
        base object.

        * gcc.target/i386/pr24178.c: New testcase.

Added:
    trunk/gcc/testsuite/gcc.target/i386/pr24178.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/emit-rtl.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug target/45212] [4.6 Regression] FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2010-08-09 11:53 -------
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

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


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

[Bug target/45212] [4.6 Regression] FAIL: gcc.target/alpha/pr24178.c scan-assembler ldl.*,18\\\\(

[ubizjak at gmail dot com]

------- Comment #5 from ubizjak at gmail dot com  2010-08-09 18:24 -------
(In reply to comment #4)
> Fixed.

Confirmed fixed on alpha [1].

[1] http://gcc.gnu.org/ml/gcc-testresults/2010-08/msg00932.html


-- 


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