This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: GCC build of HEAD failed for native with your patch on 2005-06-02T16:33:47Z.


> On Thu, Jun 02, 2005 at 05:52:33PM +0000, GCC regression checker wrote:
> 
> > --- /Users/regress/tbox/changelog_mail/gcc/gcc/ChangeLog	Wed Jun  1 15:43:56 2005
> > +++ gcc/gcc/ChangeLog	Thu Jun  2 09:34:59 2005
> > @@ -1,3 +1,171 @@
> > +2005-06-02  Dorit Nuzman  <dorit@il.ibm.com>
> > +
> > +	PR tree-optimization/21734
> > +	* tree-vectorizer.c (slpeel_update_phis_for_duplicate_loop): Use the
> > +	phi_result when current_def is not available.
> > +	(slpeel_update_phi_nodes_for_guard1): Don't fail if current_def is not
> > +	available.	
> > +
> > +2005-06-02  David Edelsohn  <edelsohn@gnu.org>
> > +
> > +	* config/rs6000/rs6000.c (rs6000_insn_valid_within_doloop): New.
> > +	(TARGET_INSN_VALID_WITHIN_DOLOOP): Define.
> > +
> > +2005-06-02  Diego Novillo  <dnovillo@redhat.com>
> > +
> > +	* tree-vrp.c (has_assert_expr, maybe_add_assert_expr): Remove.
> > +
> > +2005-06-02  Jan Hubicka  <jh@suse.cz>
> > +
> > +	* cgraph.c (dump_cgraph_node): Print new flags.
> > +	(dump_cgraph_varpool_node): Likewise.
> > +	(decide_variable_is_needed): Initialize externally_visible flag.
> > +	* cgraph.h (cgraph_local_info): Add externally_visible flag.
> > +	(cgraph_varpool_node): Likewise.
> > +	(cgraph_function_flags_ready): Declare.
> > +	* cgraph.c (cgraph_mark_local_functions): Rename to ...
> > +	(cgraph_function_and_variable_visibility) ... this one; handle
> > +	externally_visible flags.
> > +	(decide_is_function_needed): Set externally_visible flag.
> > +	(cgraph_finalize_function): Deal properly with early cleanups.
> > +	(cgraph_optimize): Update call of
> > +	cgraph_function_and_variable_visibility.
> > +
> > +2005-06-02 Steven Bosscher  <stevenb@suse.de>
> > +	   Mostafa Hagog <mustafa@il.ibm.com>
> > +
> > +	* cfgloop.h (doloop_condition_get): Make external.
> > +	* loop-doloop.c (doloop_condition_get): Generalize to make it
> > +	usable in modulo-sched.c.
> > +	* modulo-sched.c (doloop_register_get): Use
> > +	doloop_condition_get instead of duplicating it.
> > +
> > +2005-06-02  Bernd Schmidt  <bernd.schmidt@analog.com>
> > +
> > +	* reload1.c (reload): Undo 2005-04-20 change.  Make sure we detect
> > +	the correct set of init_insns that need deletion.
> > +	* local-alloc.c (update_equiv_regs): When substituting sole definition
> > +	into sole use of a reg, delete it from liveness information.
> > +
> > +2005-06-02  Kaz Kojima  <kkojima@gcc.gnu.org>
> > +
> > +	* config/sh/sh.h (TARGET_FPU_DOUBLE): Use MASK_SH4.
> > +
> 
> My bootstraps on ppc64 broke sometime today.  One of the patches
> above seems to have triggered the failure.

Testing with my patches seemed to pass fluently, but while re-reading I
spot rather ugly typo that might cause some of target dependent breakage
(as local functions might change calling convention, interestingly x86
is one of platforms that should break rather than pass with this sort of
bug). I am commiting the attached fix as obvious and hope it to solve
the particular problem.
My apologizes for breakage... 

Honza

	* cgraphunit.c (cgraph_function_and_variable_visibility): Fix typo in
	previous patch.

Index: cgraphunit.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/cgraphunit.c,v
retrieving revision 1.111
diff -c -3 -p -r1.111 cgraphunit.c
*** cgraphunit.c	2 Jun 2005 10:59:00 -0000	1.111
--- cgraphunit.c	2 Jun 2005 18:33:30 -0000
*************** cgraph_function_and_variable_visibility 
*** 1046,1052 ****
  	node->local.externally_visible = 1;
        node->local.local = (!node->needed
  			   && node->analyzed
! 			   && node->local.externally_visible);
      }
    for (vnode = cgraph_varpool_nodes_queue; vnode; vnode = vnode->next_needed)
      {
--- 1046,1052 ----
  	node->local.externally_visible = 1;
        node->local.local = (!node->needed
  			   && node->analyzed
! 			   && !node->local.externally_visible);
      }
    for (vnode = cgraph_varpool_nodes_queue; vnode; vnode = vnode->next_needed)
      {


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]