Documenation in ifcvt.c

[Richard Henderson]

On Tue, Jul 15, 2003 at 08:11:56PM -0400, Richard Kenner wrote:
> The bug I'm looking at is present on the mainline and is fixed by the
> following (so far untested) patch:

Hum.  Looks like Jan's patch isn't installed yet.  

I don't seem to be able to contact gcc.gnu.org at the moment,
so I can't search the archives for the relevant patch.  I'll
try again later this evening.

> -   return emit_store_flag (x, code, XEXP (cond, 0),
> - 			  XEXP (cond, 1), VOIDmode,
> +   return emit_store_flag (x, code, copy_rtx (XEXP (cond, 0)),
> + 			  copy_rtx (XEXP (cond, 1)), VOIDmode,
>   			  (code == LTU || code == LEU
>   			   || code == GEU || code == GTU), normalize);

The patch that I approved avoids sprinkling lots and lots
of copy_rtx calls all over the place.  I rejected that idea
because it's very error-prone to miss a place that would 
need such attention.


r~