[Bug target/12081] Gcc can't be compiled with -mregparm=3

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[stefan.kristiansson at saunalahti dot fi]

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

Stefan Kristiansson <stefan.kristiansson at saunalahti dot fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |stefan.kristiansson@saunala
                   |                            |hti.fi

--- Comment #19 from Stefan Kristiansson <stefan.kristiansson at saunalahti dot fi> ---
Created attachment 30186
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30186&action=edit
Updated version of Rask Ingemann Lambertsen's patch

I'm bumping this bug to make people aware that it still exists and that people
are affected by it.
On OpenRISC (not in main tree arch), the ABI states that var args are always
passed on the stack, thus making it impossible to build a working native
compiler
when this bug is present.

Attached is an updated version of the patch posted by Rask Ingemann Lambertsen,
bringing it up to date with the current svn.

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[mikpe at it dot uu.se]

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

Mikael Pettersson <mikpe at it dot uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mikpe at it dot uu.se

--- Comment #20 from Mikael Pettersson <mikpe at it dot uu.se> ---
Created attachment 30188
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30188&action=edit
fix up conversion in tilepro_expand_builtin, delete unused 11-arity stuff

Looking through the patch I noticed that the conversion in
tilepro_expand_builtin was broken (icode was lost).  Grepping around I also
noticed that nothing used GEN_FCN11 (or is that used by the out-of-tree
OpenRISC port?)  This add-on fixes those two issues.

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[mikpe at it dot uu.se]

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

--- Comment #21 from Mikael Pettersson <mikpe at it dot uu.se> ---
(In reply to Mikael Pettersson from comment #20)
> Grepping around I also
> noticed that nothing used GEN_FCN11 (or is that used by the out-of-tree
> OpenRISC port?)  This add-on fixes those two issues.

i386/sync.md appears to generate code that needs the 11-arity version; consider
that part of the fixup patch revoked.

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[mikpe at it dot uu.se]

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

--- Comment #22 from Mikael Pettersson <mikpe at it dot uu.se> ---
FWIW, the updated patch for gcc 4.9 bootstraps and regtests cleanly on several
hosts (x86_64, sparc64, powerpc64, armv5tel, m68k).

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[ubizjak at gmail dot com]

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

--- Comment #23 from Uroš Bizjak <ubizjak at gmail dot com> ---
(In reply to Mikael Pettersson from comment #22)
> FWIW, the updated patch for gcc 4.9 bootstraps and regtests cleanly on
> several hosts (x86_64, sparc64, powerpc64, armv5tel, m68k).

Please post the patch to gcc-patches@ mailing list for a review.
>From gcc-bugs-return-423873-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Wed Jun 05 18:41:43 2013
Return-Path: <gcc-bugs-return-423873-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 22688 invoked by alias); 5 Jun 2013 18:41:43 -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 22666 invoked by uid 48); 5 Jun 2013 18:41:40 -0000
From: "paolo.carlini at oracle dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/57472] internal compiler error: in finish_member_declaration, at cp/semantics.c
Date: Wed, 05 Jun 2013 18:41:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c++
X-Bugzilla-Version: 4.8.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: paolo.carlini at oracle dot com
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: bug_status cf_reconfirmed_on everconfirmed
Message-ID: <bug-57472-4-jnSzjeNynr@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-57472-4@http.gcc.gnu.org/bugzilla/>
References: <bug-57472-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: 2013-06/txt/msg00252.txt.bz2
Content-length: 506

http://gcc.gnu.org/bugzilla/show_bug.cgi?idW472

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2013-06-05
     Ever confirmed|0                           |1

--- Comment #1 from Paolo Carlini <paolo.carlini at oracle dot com> ---
Confirmed on x86_64-linux.

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[stefan.kristiansson at saunalahti dot fi]

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

--- Comment #24 from Stefan Kristiansson <stefan.kristiansson at saunalahti dot fi> ---
(In reply to Uroš Bizjak from comment #23)
> (In reply to Mikael Pettersson from comment #22)
> > FWIW, the updated patch for gcc 4.9 bootstraps and regtests cleanly on
> > several hosts (x86_64, sparc64, powerpc64, armv5tel, m68k).
> 
> Please post the patch to gcc-patches@ mailing list for a review.

Done: http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00400.html
>From gcc-bugs-return-426067-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Wed Jul 10 02:13:58 2013
Return-Path: <gcc-bugs-return-426067-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 26162 invoked by alias); 10 Jul 2013 02:13:58 -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 26125 invoked by uid 48); 10 Jul 2013 02:13:55 -0000
From: "hp at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug regression/57766] [4.9 Regression]: gcc.sum c-c++-common/cilk-plus/AN/sec_implicit_ex.c
Date: Wed, 10 Jul 2013 02:13:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: regression
X-Bugzilla-Version: unknown
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: hp at gcc dot gnu.org
X-Bugzilla-Status: RESOLVED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: bug_status resolution
Message-ID: <bug-57766-4-9w1ZLrONRC@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-57766-4@http.gcc.gnu.org/bugzilla/>
References: <bug-57766-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: 2013-07/txt/msg00574.txt.bz2
Content-length: 588

http://gcc.gnu.org/bugzilla/show_bug.cgi?idW766

Hans-Peter Nilsson <hp at gcc dot gnu.org> changed:

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

--- Comment #3 from Hans-Peter Nilsson <hp at gcc dot gnu.org> ---
(In reply to Balaji V. Iyer from comment #2)
> This issue should be fixed in SVN revision 200581.
>
> Thanks,
>
> Balaji V. Iyer.

Confirmed; fixed at 200581.  Thanks.

[Bug target/12081] Gcc can't be compiled with -mregparm=3

[olegendo at gcc dot gnu.org]

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

Oleg Endo <olegendo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |olegendo at gcc dot gnu.org

--- Comment #25 from Oleg Endo <olegendo at gcc dot gnu.org> ---
I've proposed a simpler patch which doesn't require fixing up backend code etc.

http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01315.html

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[olegendo at gcc dot gnu.org]

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

Oleg Endo <olegendo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|target                      |other

--- Comment #26 from Oleg Endo <olegendo at gcc dot gnu.org> ---
I don't think this is a 'target' issue, changed to 'other'.

I've just committed the following, which should fix the issue for 4.9.

http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201513

    PR other/12081
    * recog.h (rtx (*insn_gen_fn) (rtx, ...)): Replace typedef with    new
    class insn_gen_fn.
    * expr.c (move_by_pieces_1, store_by_pieces_2): Replace argument
    rtx (*) (rtx, ...) with insn_gen_fn.
    * genoutput.c (output_insn_data): Cast gen_? function pointers to
    insn_gen_fn::stored_funcptr.  Add initializer braces.


Since 4.8 also uses C++ it could be backported.

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[olegendo at gcc dot gnu.org]

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

--- Comment #31 from Oleg Endo <olegendo at gcc dot gnu.org> ---
Author: olegendo
Date: Thu Aug 29 18:37:46 2013
New Revision: 202083

URL: http://gcc.gnu.org/viewcvs?rev=202083&root=gcc&view=rev
Log:
    Backport from mainline
    2013-08-05  Oleg Endo  <olegendo@gcc.gnu.org>

    PR other/12081
    * recog.h (rtx (*insn_gen_fn) (rtx, ...)): Replace typedef with    new
    class insn_gen_fn.
    * expr.c (move_by_pieces_1, store_by_pieces_2): Replace argument
    rtx (*) (rtx, ...) with insn_gen_fn.
    * genoutput.c (output_insn_data): Cast gen_? function pointers to
    insn_gen_fn::stored_funcptr.  Add initializer braces.

    Backport from mainline
    2013-08-07  Oleg Endo  <olegendo@gcc.gnu.org>

    PR other/12081
    * config/rs6000/rs6000.c (gen_2arg_fn_t): Remove typedef.
    (rs6000_emit_swdiv_high_precision, rs6000_emit_swdiv_low_precision,
    rs6000_emit_swrsqrt): Don't cast result of GEN_FCN to gen_2arg_fn_t.


Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/config/rs6000/rs6000.c
    branches/gcc-4_8-branch/gcc/expr.c
    branches/gcc-4_8-branch/gcc/genoutput.c
    branches/gcc-4_8-branch/gcc/recog.h

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[olegendo at gcc dot gnu.org]

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

Oleg Endo <olegendo at gcc dot gnu.org> changed:

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

--- Comment #32 from Oleg Endo <olegendo at gcc dot gnu.org> ---
Fixed on 4.8 and 4.9.  Won't backport to 4.7 since it doesn't use C++.

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[oyvind.harboe at zylin dot com]

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

oyvind.harboe at zylin dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |oyvind.harboe at zylin dot com

--- Comment #33 from oyvind.harboe at zylin dot com ---
This may be of interest historically: The SPEC2006 403.gcc benchmark has this
bug. 403.gcc is based on gcc 3.2.

https://www.spec.org/cpu2006/Docs/403.gcc.html

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[oyvind.harboe at zylin dot com]

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

--- Comment #34 from oyvind.harboe at zylin dot com ---
SPEC2017 502.gcc_r is based on gcc 4.5.0, which also has this bug.

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[oyvind.harboe at zylin dot com]

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

--- Comment #35 from oyvind.harboe at zylin dot com ---
SPEC 2017 added SPEC_GCC_VARIADIC_FUNCTIONS_MISMATCH_WORKAROUND to cope with
this error.

[Bug other/12081] Gcc can't be compiled with -mregparm=3

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|middle-end                  |other