AIX regression due to DFA scheduler merge

law@redhat.com law@redhat.com
Mon Jun 10 10:38:00 GMT 2002


In message <200206062239.09774@enzo.bigblue.local>, Franz Sirl writes:
 > 
 > --------------Boundary-00=_9DXA4ANF4OWQ32G4SHO5
 > Content-Type: text/plain;
 >   charset="iso-8859-1"
 > Content-Transfer-Encoding: 8bit
 > 
 > On Tuesday 04 June 2002 23:24, Franz Sirl wrote:
 > > On Tuesday 04 June 2002 21:55, Janis Johnson wrote:
 > > > On Tue, Jun 04, 2002 at 07:51:43PM +0200, Franz Sirl wrote:
 > > > > OK to commit?
 > > > >
 > > > > Franz.
 > > > >
 > > > > 	* rtl.h (SCHED_GROUP_P): Disallow CODE_LABEL, BARRIER and NOTE.
 > > > > 	* haifa-sched.c (move_insn): Adjust for the SCHED_GROUP_P chang
 > e.
 > > > > 	(set_priorities): Likewise.
 > > > > 	* sched-deps.c (add_dependence): Likewise:
 > > > > 	(group_leader): Likewise.
 > > > > 	* sched-ebb.c (init_ready_list): Likewise.
 > > > > 	* sched-rgn.c (init_ready_list): Likewise.
 > > >
 > > > The RTL flags are documented in rtl.texi, so please update the entries
 > > > for the modified macro there as well (once for the macro and once for
 > > > the flag).
 > >
 > > Like that? I fixed the /i -> /s typo on the way.
 > >
 > > Franz.
 > >
 > >
 > > 	* rtl.h (SCHED_GROUP_P): Disallow CODE_LABEL, BARRIER and NOTE.
 > > 	* haifa-sched.c (move_insn): Adjust for the SCHED_GROUP_P change.
 > > 	(set_priorities): Likewise.
 > > 	* sched-deps.c (add_dependence): Likewise:
 > > 	(group_leader): Likewise.
 > > 	* sched-ebb.c (init_ready_list): Likewise.
 > > 	* sched-rgn.c (init_ready_list): Likewise.
 > > 	* doc/rtl.texi: Adjust accordingly.
 > 
 > It turned out the hunks in haifa-sched.c and sched-ebb.c are unnecessary for
 >  a 
 > successful regressionless bootstrap on powerpc-linux-gnu and i686-linux-gnu.
 > 
 > Jeff, OK to commit to mainline?
 > 
 > Franz.
 > 
 > 	* rtl.h (SCHED_GROUP_P): Disallow CODE_LABEL, BARRIER and NOTE.
 > 	* sched-deps.c (add_dependence): Likewise.
 > 	(group_leader): Likewise.
 > 	* sched-rgn.c (init_ready_list): Likewise.
 > 	* doc/rtl.texi: Adjust accordingly.
Yes, please go ahead and install this change.
jeff



More information about the Gcc-patches mailing list