[Bug lto/64795] New: [5.0 regression x86_64] too many memory references for `lea'

[Bug lto/64795] New: [5.0 regression x86_64] too many memory references for `lea'

[dimhen at gmail dot com]

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

            Bug ID: 64795
           Summary: [5.0 regression x86_64] too many memory references for
                    `lea'
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: lto
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dimhen at gmail dot com

r218766 PASS
r218767 FAIL
r220090 FAIL

Fedora 21 / x86_64

$ cat t.ii
typedef void tf(struct s);
struct s {
  tf *pf;
};
void fn1(void *);
static void fn2(s);
main() {
  static s a;
  fn1(&a);
  a.pf = fn2;
}

$ touch r.ii
$ /home/dimhen/bin/gcc_218767_yes/bin/g++ -fpreprocessed -O -flto -fPIC -DPIC
-c -o r.o r.ii
$ /home/dimhen/bin/gcc_218767_yes/bin/g++ -fpreprocessed -O -flto -c -o t.o
t.ii
$ $ /home/dimhen/bin/gcc_218767_yes/bin/g++ -flto -o x t.o r.o -save-temps
x.ltrans0.s: Assembler messages:
x.ltrans0.s:12: Error: too many memory references for `lea'
lto-wrapper: fatal error: /home/dimhen/bin/gcc_218767_yes/bin/g++ returned 1
exit status
compilation terminated.
/usr/bin/ld: fatal error: lto-wrapper failed
collect2: error: ld returned 1 exit status

$ cat x.ltrans0.s
    .file    "x.ltrans0.o"
    .text
    .globl    main
    .type    main, @function
main:
.LFB0:
    .cfi_startproc
    subq    $8, %rsp
    .cfi_def_cfa_offset 16
    leaq    _ZZ4mainE1a(%rip), %rdi
    call    _Z3fn1Pv@PLT
    leaq    _ZL3fn21s(%rip), _ZZ4mainE1a(%rip)
    movl    $0, %eax
    addq    $8, %rsp
    .cfi_def_cfa_offset 8
    ret
    .cfi_endproc
.LFE0:
    .size    main, .-main
    .local    _ZZ4mainE1a
    .comm    _ZZ4mainE1a,8,8
    .ident    "GCC: (GNU) 5.0.0 20141215 (experimental) [trunk revision
218767]"
    .section    .note.GNU-stack,"",@progbits

$ /home/dimhen/bin/gcc_218767_yes/bin/g++ -v                     
Using built-in specs.
COLLECT_GCC=/home/dimhen/bin/gcc_218767_yes/bin/g++
COLLECT_LTO_WRAPPER=/home/dimhen/bin/gcc_218767_yes/bin/../libexec/gcc/x86_64-unknown-linux-gnu/5.0.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /home/dimhen/src/gcc_r218767/configure
--prefix=/usr/local/gcc_current --enable-static --enable-checking=yes
--enable-languages=c,c++,lto --enable-plugin --disable-libstdcxx-dual-abi
Thread model: posix
gcc version 5.0.0 20141215 (experimental) [trunk revision 218767] (GCC)

[Bug lto/64795] [5.0 regression x86_64] too many memory references for `lea'

[dimhen at gmail dot com]

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

--- Comment #1 from Dmitry G. Dyachenko <dimhen at gmail dot com> ---
possible duplicate PR64374 ?

[Bug target/64795] [5.0 regression x86_64] too many memory references for `lea'

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |lto
             Target|                            |x86_64-*-*
          Component|lto                         |target
   Target Milestone|---                         |5.0

[Bug target/64795] [5 regression] too many memory references for `lea'

[uros at gcc dot gnu.org]

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

--- Comment #3 from uros at gcc dot gnu.org ---
Author: uros
Date: Mon Jan 26 18:49:21 2015
New Revision: 220128

URL: https://gcc.gnu.org/viewcvs?rev=220128&root=gcc&view=rev
Log:
        PR target/64795
    * config/i386/i386.md (*movdi_internal): Also check operand 0
    to determine TYPE_LEA operand.
    (*movsi_internal): Ditto.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.md

[Bug target/64795] [5 regression] too many memory references for `lea'

[uros at gcc dot gnu.org]

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

--- Comment #4 from uros at gcc dot gnu.org ---
Author: uros
Date: Mon Jan 26 20:12:26 2015
New Revision: 220132

URL: https://gcc.gnu.org/viewcvs?rev=220132&root=gcc&view=rev
Log:
    Backport from mainline
    2015-01-26  Uros Bizjak  <ubizjak@gmail.com>

    PR target/64795
    * config/i386/i386.md (*movdi_internal): Also check operand 0
    to determine TYPE_LEA operand.
    (*movsi_internal): Ditto.

    Backport from mainline
    2015-01-23  Uros Bizjak  <ubizjak@gmail.com>

    * config/i386/sse.md (sse2_loadld): Set attribute isa to sse2 for
    alternative 1.


Modified:
    branches/gcc-4_9-branch/gcc/ChangeLog
    branches/gcc-4_9-branch/gcc/config/i386/i386.md
    branches/gcc-4_9-branch/gcc/config/i386/sse.md

[Bug target/64795] [5 regression] too many memory references for `lea'

[uros at gcc dot gnu.org]

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

--- Comment #5 from uros at gcc dot gnu.org ---
Author: uros
Date: Mon Jan 26 20:49:22 2015
New Revision: 220134

URL: https://gcc.gnu.org/viewcvs?rev=220134&root=gcc&view=rev
Log:
    Backport from mainline
    2015-01-26  Uros Bizjak  <ubizjak@gmail.com>

    PR target/64795
    * config/i386/i386.md (*movdi_internal): Also check operand 0
    to determine TYPE_LEA operand.
    (*movsi_internal): Ditto.

    Backport from mainline
    2015-01-23  Uros Bizjak  <ubizjak@gmail.com>

    * config/i386/sse.md (sse2_loadld): Set attribute isa to sse2 for
    alternative 1.


Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/config/i386/i386.md
    branches/gcc-4_8-branch/gcc/config/i386/sse.md

[Bug target/64795] [5 regression] too many memory references for `lea'

[ubizjak at gmail dot com]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 3070 bytes --]

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

Uroš Bizjak <ubizjak at gmail dot com> changed:

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

--- Comment #6 from Uroš Bizjak <ubizjak at gmail dot com> ---
Fixed everywhere.
>From gcc-bugs-return-474965-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Mon Jan 26 20:57:20 2015
Return-Path: <gcc-bugs-return-474965-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 30603 invoked by alias); 26 Jan 2015 20:57:19 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 30535 invoked by uid 48); 26 Jan 2015 20:57:12 -0000
From: "rth at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug libffi/64799] [5 regression] libffi.special/unwindtest.cc FAILs on Solaris/SPARC
Date: Mon, 26 Jan 2015 20:57:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: libffi
X-Bugzilla-Version: 5.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: rth at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: rth at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 5.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: bug_status cf_reconfirmed_on assigned_to everconfirmed attachments.created
Message-ID: <bug-64799-4-I5vxoQ4ma1@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-64799-4@http.gcc.gnu.org/bugzilla/>
References: <bug-64799-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2015-01/txt/msg02959.txt.bz2
Content-length: 773

https://gcc.gnu.org/bugzilla/show_bug.cgi?idd799

Richard Henderson <rth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-01-26
           Assignee|unassigned at gcc dot gnu.org      |rth at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Richard Henderson <rth at gcc dot gnu.org> ---
Created attachment 34583
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id4583&actioníit
proposed patch

That does seem likely.  I tested this on linux; hopefully
I got all of the macroization right for solaris.