Re: Your proposed change to loop.c

[Mark Mitchell <mark at codesourcery dot com>]

>>>>> "Joern" == Joern Rennecke <amylaar@cygnus.co.uk> writes:

    Joern> No.  Getting into any more detail would mean making a
    Joern> patch, but I don't have any spare CPU cycles now to test
    Joern> it.

Since you're unable to help out at the moment, I've backed out the
changes that are causing the trouble.  I bet they fix bugs and provide
real improvements, so I hope that you'll take the time to explain them
to someone so that they can go back in.  I'll be happy to help with
that if I can.

I note that those patches were not accompanied by either commentary or
test-cases in the mail messages you sent when you checked them in, so
it's hard to figure out what they were for.  I suggest resubmitting
them, in well-tested chunks, with some explanation.

I've verified that the mainline tree returns to bootstrap-ville with
these patches out.

Thanks,

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com

Thu Jun 24 15:00:47 1999  Mark Mitchell  <mark@codesourcery.com>

	Revert these two patches:

	Thu Jun 17 21:34:24 1999  J"orn Rennecke <amylaar@cygnus.co.uk>
	* loop.c (strength_reduce): When doing biv->giv conversion, update
	reg note of NEXT->insn.

	Thu Jun 17 17:22:07 1999  J"orn Rennecke <amylaar@cygnus.co.uk>
	* loop.c (strength_reduce): When doing biv->giv conversion, fix up
	reg_biv_class.
	(recombine_givs): Set ix field after sorting.