This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch/committed] PR middle-end/65233 make walk-ssa_copies handle empty PHIs
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Richard Biener <richard dot guenther at gmail dot com>
- Cc: Jeff Law <law at redhat dot com>, Aldy Hernandez <aldyh at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Thu, 5 Mar 2015 01:54:44 +0100
- Subject: Re: [patch/committed] PR middle-end/65233 make walk-ssa_copies handle empty PHIs
- Authentication-results: sourceware.org; auth=none
- References: <54F3F648 dot 8090400 at redhat dot com> <CAFiYyc3qQhXgkLB_zgO9fOmqQ0caSpxGTCr6ERy4VJvX4H9jkw at mail dot gmail dot com> <54F697A5 dot 6090908 at redhat dot com> <CAFiYyc0FWK45geB3qKoj-RjQM3mPh3+d3o_Kuh+KZsBrzBUskQ at mail dot gmail dot com> <CAFiYyc0k5pkR8ju+VM1G6FNvg2xkjRwPEu6XtZuqy-uENc3GYQ at mail dot gmail dot com>
> >
> > It gets passed the valueize callback now which returns NULL_TREE for
> > SSA names we can't follow.
>
> Btw, for match-and-simplify I had to use that as default for fold_stmt
> _exactly_ because of the call to fold_stmt from replace_uses_by
> via merge-blocks from cfgcleanup. This is because replace-uses-by
> doesn't have all uses replaced before it folds the stmt!
>
> We also have the "weaker" in-place flag.
>
> 2015-03-04 Richard Biener <rguenther@suse.de>
>
> PR middle-end/65233
> * ipa-polymorphic-call.c: Include tree-ssa-operands.h and
> tree-into-ssa.h.
> (walk_ssa_copies): Revert last chage. Instead do not walk
> SSA names registered for SSA update.
Maybe include the patch? It should not be problem to make the function
to valuelize everything it looks into.
Honza