This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [tree-ssa]: PRE updates
- From: Andrew MacLeod <amacleod at redhat dot com>
- To: Daniel Berlin <dberlin at dberlin dot org>
- Cc: Diego Novillo <dnovillo at redhat dot com>,gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: 11 Jun 2003 10:51:51 -0400
- Subject: Re: [tree-ssa]: PRE updates
- References: <B716EC02-9C19-11D7-870D-000A95A34564@dberlin.org>
On Wed, 2003-06-11 at 10:33, Daniel Berlin wrote:
>
> On Wednesday, June 11, 2003, at 10:04 AM, Andrew MacLeod wrote:
>
> > On Wed, 2003-06-11 at 09:49, Diego Novillo wrote:
> >> On Wed, 2003-06-11 at 09:27, Andrew MacLeod wrote:
> > Dan, whats the line in your source code that is failing, and if its in
> > that hunk, whats the PHi node? the one I just created?
>
>
> I was about to say.
> The place in *my* source it's failing on is:
> /* Assign root variable as partition representative for each live on
> entry
> partition. */
> EXECUTE_IF_SET_IN_SBITMAP (live, 0, x,
> {
> var = root_var (rv, find_root_var (rv, x));
> ann = var_ann (var);
> /* If these aren't already coalesced... */
> if (partition_to_var (map, x) != var)
> {
> if (ann->out_of_ssa_tag)
> {
> /* This root variable has already been assigned to another
> partition which is not coalesced with this one. */
> abort ();
> }
>
> if (tree_ssa_dump_file && (tree_ssa_dump_flags & TDF_DETAILS))
> {
> fprintf (tree_ssa_dump_file, "Must coalesce ");
> print_generic_expr (tree_ssa_dump_file,
> partition_to_var (map, x),
> TDF_SLIM);
> fprintf (tree_ssa_dump_file, " with the root variable ");
> print_generic_expr (tree_ssa_dump_file, var, TDF_SLIM);
> fprintf (tree_ssa_dump_file, ".\n");
> }
>
> change_partition_var (map, var, x);
> }
> });
>
> (inside the macro, so it gets tagged as the end of the macro).
>
> There is only one abort in this code, so i assume that's it.
>
maybe... I guess I'll have to apply your patch and look at it myself.
Andrew