Regression in unroll.c

Jeffrey A Law law@cygnus.com
Wed Jun 10 00:53:00 GMT 1998


  In message < 199806081018.LAA26822@sun52.NIS.cambridge >you write:
  > The gcc2 change of Mar 21 to verify_addresses (and merged into egcs) 
  > causes a regression on the ARM (execute/920625-1.c) with -funroll-loops).  
  > The problem is that verify_addresses was modified to use 
  > validate_replace_rtx so that match_dups would be handled, but the 
  > corresponding change to the undo at the end of the function wasn't made.  
  > Since the comment asserts that the undo can't fail, I've also added an 
  > abort to confirm this.
  > 
  > <date>  Richard Earnshaw (rearnsha@arm.com)
  > 
  > 	* unroll.c (verify_addresses): Use validate_replace_rtx to undo the 
  > 	changes.  Abort if the undo fails.
This looks good to me.  Please install it into egcs.

Thanks!

jeff



More information about the Gcc-patches mailing list