public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "cvs-commit at gcc dot gnu dot org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/19317] [4.1 Regression] removing a temporary return value when we cannot Date: Thu, 23 Jun 2005 14:44:00 -0000 [thread overview] Message-ID: <20050623144438.5477.qmail@sourceware.org> (raw) In-Reply-To: <20050107191510.19317.mueller@kde.org> ------- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-06-23 14:44 ------- Subject: Bug 19317 CVSROOT: /cvs/gcc Module name: gcc Changes by: jason@gcc.gnu.org 2005-06-23 14:44:21 Modified files: gcc : ChangeLog tree.h calls.c tree-inline.c tree-pretty-print.c gimplify.c tree-nrv.c tree-pass.h tree-optimize.c gcc/cp : ChangeLog semantics.c Added files: gcc/testsuite/g++.dg/opt: nrv9.C Log message: PR c++/19317 Leave the return slot target in the MODIFY_EXPR rather than making it an argument, but only use it if the CALL_EXPR has a flag set. * tree.h (CALL_EXPR_HAS_RETURN_SLOT_ADDR): Rename to CALL_EXPR_RETURN_SLOT_OPT. * calls.c (expand_call): Adjust. * tree-inline.c (expand_call_inline): Adjust. * tree-pretty-print.c (dump_generic_node): Adjust. And set the flag as appropriate. * gimplify.c (gimplify_modify_expr_rhs): Set CALL_EXPR_HAS_RETURN_SLOT_ADDR where the LHS is obviously safe. * tree-nrv.c (execute_return_slot_opt): Set CALL_EXPR_HAS_RETURN_SLOT_ADDR based on escape analysis. * tree-pass.h: Declare pass_return_slot. * tree-optimize.c (init_tree_optimization_passes): Add it. * cp/semantics.c (simplify_aggr_init_expr): Use CALL_EXPR_RETURN_SLOT_OPT, not CALL_EXPR_HAS_RETURN_SLOT_ADDR. Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.9211&r2=2.9212 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree.h.diff?cvsroot=gcc&r1=1.737&r2=1.738 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/calls.c.diff?cvsroot=gcc&r1=1.389&r2=1.390 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-inline.c.diff?cvsroot=gcc&r1=1.196&r2=1.197 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-pretty-print.c.diff?cvsroot=gcc&r1=2.63&r2=2.64 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/gimplify.c.diff?cvsroot=gcc&r1=2.136&r2=2.137 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-nrv.c.diff?cvsroot=gcc&r1=2.7&r2=2.8 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-pass.h.diff?cvsroot=gcc&r1=2.42&r2=2.43 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-optimize.c.diff?cvsroot=gcc&r1=2.108&r2=2.109 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&r1=1.4797&r2=1.4798 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/semantics.c.diff?cvsroot=gcc&r1=1.477&r2=1.478 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/opt/nrv9.C.diff?cvsroot=gcc&r1=NONE&r2=1.1 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19317
next prev parent reply other threads:[~2005-06-23 14:44 UTC|newest] Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top 2005-01-07 19:15 [Bug c++/19317] New: forgets to deep-copy temporary return value mueller at kde dot org 2005-01-07 19:16 ` [Bug c++/19317] " mueller at kde dot org 2005-01-07 19:16 ` [Bug c++/19317] [4.0 Regression] " pinskia at gcc dot gnu dot org 2005-01-07 19:26 ` pinskia at gcc dot gnu dot org 2005-01-07 19:30 ` [Bug c++/19317] [4.0 Regression] removing a temporary return value when we cannot pinskia at gcc dot gnu dot org 2005-01-10 4:52 ` pinskia at gcc dot gnu dot org 2005-01-10 5:03 ` pinskia at gcc dot gnu dot org 2005-01-15 12:45 ` steven at gcc dot gnu dot org 2005-01-24 13:27 ` pinskia at gcc dot gnu dot org 2005-02-13 21:24 ` mueller at kde dot org 2005-02-13 21:31 ` pinskia at gcc dot gnu dot org 2005-02-16 6:51 ` mueller at kde dot org 2005-03-10 23:49 ` [Bug c++/19317] [4.0/4.1 " pluto at pld-linux dot org 2005-03-17 17:33 ` mueller at kde dot org 2005-03-17 18:22 ` pluto at pld-linux dot org 2005-03-18 0:34 ` jason at redhat dot com 2005-03-24 11:07 ` pluto at pld-linux dot org 2005-03-24 12:46 ` rwgk at yahoo dot com 2005-03-24 21:04 ` pinskia at gcc dot gnu dot org 2005-04-05 6:25 ` mmitchel at gcc dot gnu dot org 2005-04-05 23:14 ` cvs-commit at gcc dot gnu dot org 2005-04-07 21:01 ` [Bug c++/19317] [4.1 " pinskia at gcc dot gnu dot org 2005-04-08 13:33 ` pluto at pld-linux dot org 2005-04-09 2:29 ` pluto at pld-linux dot org 2005-04-11 7:49 ` jason at redhat dot com 2005-04-11 9:00 ` pluto at pld-linux dot org 2005-04-11 12:48 ` jason at redhat dot com 2005-04-11 14:44 ` pluto at pld-linux dot org 2005-04-12 10:27 ` mueller at kde dot org 2005-04-12 20:01 ` mlists at juma dot me dot uk 2005-04-13 1:10 ` mueller at kde dot org 2005-04-13 1:36 ` mueller at kde dot org 2005-04-13 16:57 ` mueller at kde dot org 2005-05-17 5:41 ` bernie at develer dot com 2005-05-18 19:38 ` jason at gcc dot gnu dot org 2005-05-18 20:45 ` bernie at develer dot com 2005-05-18 21:18 ` jason at redhat dot com 2005-05-19 9:04 ` pluto at agmk dot net 2005-05-19 9:45 ` bernie at develer dot com 2005-05-19 10:42 ` bernie at develer dot com 2005-06-23 14:44 ` cvs-commit at gcc dot gnu dot org [this message] 2005-06-24 14:11 ` pinskia at gcc dot gnu dot org 2005-07-08 14:13 ` cvs-commit at gcc dot gnu dot org 2005-09-26 23:18 ` kev dot gilbert at cdu dot edu dot au 2005-09-27 0:44 ` bangerth at dealii dot org 2005-09-27 7:28 ` kev dot gilbert at cdu dot edu dot au 2005-09-27 14:01 ` bangerth at dealii dot org 2005-09-30 1:59 ` kev dot gilbert at cdu dot edu dot au [not found] <bug-19317-3563@http.gcc.gnu.org/bugzilla/> 2005-12-08 21:50 ` jakub at gcc dot gnu dot org 2005-12-08 21:56 ` jakub at gcc dot gnu dot org
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20050623144438.5477.qmail@sourceware.org \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).