diff mbox

[1/1] gcc/ira-build.c: save a conflict obj compare in ira_flattening

Message ID 1419230056-3293-1-git-send-email-yizhouzhou@ict.ac.cn
State New
Headers show

Commit Message

Zhouyi Zhou Dec. 22, 2014, 6:34 a.m. UTC
In function ira_flattening, conflict relationship will be rebuilt in case
of any new pseudo register has been created during ira_emit.

By adding the current object to OBJECTS_LIVE after traversing OBJECTS_LIVE, a conflict
obj compare can be saved.


Bootstraped and regtested in x86_64 Linux 
Signed-off-by: Zhouyi Zhou <yizhouzhou@ict.ac.cn>
---
 gcc/ChangeLog   |    5 +++++
 gcc/ira-build.c |    2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

Comments

Jeff Law Dec. 22, 2014, 4:55 p.m. UTC | #1
On 12/21/14 23:34, Zhouyi Zhou wrote:
> In function ira_flattening, conflict relationship will be rebuilt in case
> of any new pseudo register has been created during ira_emit.
>
> By adding the current object to OBJECTS_LIVE after traversing OBJECTS_LIVE, a conflict
> obj compare can be saved.
>
>
> Bootstraped and regtested in x86_64 Linux
> Signed-off-by: Zhouyi Zhou <yizhouzhou@ict.ac.cn>
> ---
>   gcc/ChangeLog   |    5 +++++
>   gcc/ira-build.c |    2 +-
>   2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> index e5d16f9..c0ce758 100644
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,8 @@
> +2014-12-22  Zhouyi Zhou <yizhouzhou@ict.ac.cn>
> +
> +	* ira-build.c (ira_flattening): Add the current
> +	object to OBJECTS_LIVE after traversing OBJECTS_LIVE.
This is OK.  Please install on the trunk.

Thanks,
Jeff
Zhouyi Zhou Dec. 23, 2014, 1:28 a.m. UTC | #2
Thanks Jeff for reviewing,
I forget to mention that I do not have write access
to gcc. 
Can you install for me ?
Cheers 
Zhouyi

> -----Original Messages-----
> From: "Jeff Law" <law@redhat.com>
> Sent Time: Tuesday, December 23, 2014
> To: "Zhouyi Zhou" <zhouzhouyi@gmail.com>, gcc-patches@gcc.gnu.org
> Cc: "Zhouyi Zhou" <yizhouzhou@ict.ac.cn>
> Subject: Re: [PATCH 1/1] gcc/ira-build.c: save a conflict obj compare in ira_flattening
> 
> On 12/21/14 23:34, Zhouyi Zhou wrote:
> > In function ira_flattening, conflict relationship will be rebuilt in case
> > of any new pseudo register has been created during ira_emit.
> >
> > By adding the current object to OBJECTS_LIVE after traversing OBJECTS_LIVE, a conflict
> > obj compare can be saved.
> >
> >
> > Bootstraped and regtested in x86_64 Linux
> > Signed-off-by: Zhouyi Zhou <yizhouzhou@ict.ac.cn>
> > ---
> >   gcc/ChangeLog   |    5 +++++
> >   gcc/ira-build.c |    2 +-
> >   2 files changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> > index e5d16f9..c0ce758 100644
> > --- a/gcc/ChangeLog
> > +++ b/gcc/ChangeLog
> > @@ -1,3 +1,8 @@
> > +2014-12-22  Zhouyi Zhou <yizhouzhou@ict.ac.cn>
> > +
> > +	* ira-build.c (ira_flattening): Add the current
> > +	object to OBJECTS_LIVE after traversing OBJECTS_LIVE.
> This is OK.  Please install on the trunk.
> 
> Thanks,
> Jeff
>
Jeff Law Dec. 23, 2014, 7:03 p.m. UTC | #3
On 12/22/14 18:28, Zhouyi Zhou wrote:
> Thanks Jeff for reviewing,
> I forget to mention that I do not have write access
> to gcc.
> Can you install for me ?
Done.  Thanks,
Jeff
diff mbox

Patch

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e5d16f9..c0ce758 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@ 
+2014-12-22  Zhouyi Zhou <yizhouzhou@ict.ac.cn>
+
+	* ira-build.c (ira_flattening): Add the current
+	object to OBJECTS_LIVE after traversing OBJECTS_LIVE.
+	
 2014-12-17  Oleg Endo  <olegendo@gcc.gnu.org>
 
 	PR target/55212
diff --git a/gcc/ira-build.c b/gcc/ira-build.c
index af82b71..66318096 100644
--- a/gcc/ira-build.c
+++ b/gcc/ira-build.c
@@ -3252,7 +3252,6 @@  ira_flattening (int max_regno_before_emit, int ira_max_point_before_emit)
 		continue;
 
 	      aclass = ALLOCNO_CLASS (a);
-	      sparseset_set_bit (objects_live, OBJECT_CONFLICT_ID (obj));
 	      EXECUTE_IF_SET_IN_SPARSESET (objects_live, n)
 		{
 		  ira_object_t live_obj = ira_object_id_map[n];
@@ -3264,6 +3263,7 @@  ira_flattening (int max_regno_before_emit, int ira_max_point_before_emit)
 		      && live_a != a)
 		    ira_add_conflict (obj, live_obj);
 		}
+	      sparseset_set_bit (objects_live, OBJECT_CONFLICT_ID (obj));
 	    }
 
 	  for (r = ira_finish_point_ranges[i]; r != NULL; r = r->finish_next)