public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jan Hubicka <hubicka@ucw.cz>
To: gcc-patches@gcc.gnu.org
Subject: PR tree-optimize/48836 (internal compiler error: in execute_todo,	at passes.c:1261)
Date: Sun, 12 Jun 2011 22:24:00 -0000	[thread overview]
Message-ID: <20110612180722.GA21062@kam.mff.cuni.cz> (raw)

Hi,
I had this patch sitting in my tree for a while, but somehow forgot to commit it.
The issue here is that edge redirection affects vops and thus imply need for
SSA update in some cases.

Bootstrapped/regtested x86_64-linux, will commit it shortly.
	PR middle-end/48836
	* ipa-inline-transform.c: Include tree-pass.h
	(inline_transform): Set TODO_update_ssa_only_virtuals.
	* Makefile.in (ipa-inline-transform.o): Add tree-pass.h.
Index: ipa-inline-transform.c
===================================================================
--- ipa-inline-transform.c	(revision 174958)
+++ ipa-inline-transform.c	(working copy)
@@ -45,6 +45,7 @@ along with GCC; see the file COPYING3.  
 #include "ipa-prop.h"
 #include "ipa-inline.h"
 #include "tree-inline.h"
+#include "tree-pass.h"
 
 int ncalls_inlined;
 int nfunctions_inlined;
@@ -338,6 +339,8 @@ inline_transform (struct cgraph_node *no
       cgraph_redirect_edge_call_stmt_to_callee (e);
       if (!e->inline_failed || warn_inline)
         inline_p = true;
+      /* Redirecting edges might lead to a need for vops to be recomputed.  */
+      todo |= TODO_update_ssa_only_virtuals;
     }
 
   if (inline_p)
Index: Makefile.in
===================================================================
--- Makefile.in	(revision 174958)
+++ Makefile.in	(working copy)
@@ -3087,7 +3087,7 @@ ipa-inline-transform.o : ipa-inline-tran
    $(TREE_H) langhooks.h $(TREE_INLINE_H) $(FLAGS_H) $(CGRAPH_H) intl.h \
    $(DIAGNOSTIC_H) $(PARAMS_H) $(TIMEVAR_H) $(TREE_PASS_H) \
    $(HASHTAB_H) $(COVERAGE_H) $(GGC_H) $(TREE_FLOW_H) $(IPA_PROP_H) \
-   gimple-pretty-print.h ipa-inline.h $(LTO_STREAMER_H)
+   gimple-pretty-print.h ipa-inline.h $(LTO_STREAMER_H) tree-pass.h
 ipa-utils.o : ipa-utils.c $(IPA_UTILS_H) $(CONFIG_H) $(SYSTEM_H) \
    coretypes.h $(TM_H) $(TREE_H) $(TREE_FLOW_H) $(TREE_INLINE_H) langhooks.h \
    pointer-set.h $(GGC_H) $(GIMPLE_H) $(SPLAY_TREE_H) \

                 reply	other threads:[~2011-06-12 18:07 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20110612180722.GA21062@kam.mff.cuni.cz \
    --to=hubicka@ucw.cz \
    --cc=gcc-patches@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: link
Be 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).