public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "hubicka at ucw dot cz" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/19241] [4.0 Regression] ICE in make_decl_rtl with inliner Date: Wed, 12 Jan 2005 23:47:00 -0000 [thread overview] Message-ID: <20050112234706.2186.qmail@sourceware.org> (raw) In-Reply-To: <20050103134043.19241.larsbj@gullik.net> ------- Additional Comments From hubicka at ucw dot cz 2005-01-12 23:47 ------- Subject: Re: [4.0 Regression] ICE in make_decl_rtl with inliner Hi, the problem is retval being mapped to &temporary. When copy_body_p replaces retval by this expression, the addr_expr is walked recursively and temporary is not in the decl map so it is remapped to newly allocated decls that never gets placed in bind_expr. I think we either can insert map temporary == temporary when creating the expression or avoid recursing when remapping happens. The seconds sounds cheaper and I think it is valid, so I am testing the attached fix. I am not familiar with inliner implementation enought to say for sure that we never recurse into the replacements for reason tought. Index: tree-inline.c =================================================================== RCS file: /cvs/gcc/gcc/gcc/tree-inline.c,v retrieving revision 1.162 diff -c -3 -p -r1.162 tree-inline.c *** tree-inline.c 4 Jan 2005 01:54:25 -0000 1.162 --- tree-inline.c 12 Jan 2005 23:43:04 -0000 *************** copy_body_r (tree *tp, int *walk_subtree *** 503,508 **** --- 503,509 ---- /* Replace this variable with the copy. */ STRIP_TYPE_NOPS (new_decl); *tp = new_decl; + *walk_subtrees = 0; } else if (TREE_CODE (*tp) == STATEMENT_LIST) copy_statement_list (tp); -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19241
next prev parent reply other threads:[~2005-01-12 23:47 UTC|newest] Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top 2005-01-03 13:41 [Bug c++/19241] New: ICE in make_decl_rtl, at varasm.c:867 larsbj at gullik dot net 2005-01-03 15:28 ` [Bug c++/19241] " pinskia at gcc dot gnu dot org 2005-01-03 16:22 ` larsbj at gullik dot net 2005-01-03 18:04 ` pinskia at gcc dot gnu dot org 2005-01-03 18:25 ` larsbj at gullik dot net 2005-01-03 19:03 ` pinskia at gcc dot gnu dot org 2005-01-03 19:21 ` larsbj at gullik dot net 2005-01-03 19:38 ` [Bug c++/19241] [4.0 Regression] " pinskia at gcc dot gnu dot org 2005-01-03 19:56 ` larsbj at gullik dot net 2005-01-03 20:08 ` pinskia at gcc dot gnu dot org 2005-01-03 20:19 ` larsbj at gullik dot net 2005-01-03 20:22 ` larsbj at gullik dot net 2005-01-03 20:28 ` larsbj at gullik dot net 2005-01-03 20:41 ` pinskia at gcc dot gnu dot org 2005-01-03 20:46 ` larsbj at gullik dot net 2005-01-04 19:48 ` pinskia at gcc dot gnu dot org 2005-01-05 17:31 ` arend dot bayer at web dot de 2005-01-05 18:14 ` larsbj at gullik dot net 2005-01-05 18:53 ` arend dot bayer at web dot de 2005-01-05 18:58 ` arend dot bayer at web dot de 2005-01-06 18:28 ` [Bug tree-optimization/19241] " pinskia at gcc dot gnu dot org 2005-01-06 18:55 ` nathan at gcc dot gnu dot org 2005-01-07 12:09 ` [Bug tree-optimization/19241] [4.0 Regression] ICE in make_decl_rtl with inliner nathan at gcc dot gnu dot org 2005-01-08 22:54 ` pinskia at gcc dot gnu dot org 2005-01-09 5:01 ` pinskia at gcc dot gnu dot org 2005-01-09 7:29 ` aj at gcc dot gnu dot org 2005-01-09 16:06 ` hubicka at ucw dot cz 2005-01-09 16:08 ` pinskia at gcc dot gnu dot org 2005-01-09 16:20 ` pinskia at gcc dot gnu dot org 2005-01-11 15:29 ` pinskia at gcc dot gnu dot org 2005-01-12 11:45 ` steven at gcc dot gnu dot org 2005-01-12 11:51 ` steven at gcc dot gnu dot org 2005-01-12 23:47 ` hubicka at ucw dot cz [this message] 2005-01-13 0:11 ` jason at redhat dot com 2005-01-25 23:02 ` pinskia at gcc dot gnu dot org 2005-01-26 9:21 ` steven at gcc dot gnu dot org 2005-02-24 13:56 ` cvs-commit 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=20050112234706.2186.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).