Re: Puzzled by a recent patch

[Richard Henderson <rth at cygnus dot com>]

On Wed, Sep 06, 2000 at 04:46:13PM +0100, Bernd Schmidt wrote:
> I'm puzzled by the change below which you made in June.  More specifically,
> I don't quite follow what you are trying to do by calling
> smallest_mode_for_size.

I wanted a mode equivalent to the size computed before 

> -                       == GET_MODE_SIZE (mode) + word * UNITS_PER_WORD)

but clearly I out-foxed myself and screwed it up.

Looks like you want

-		  need_mode = smallest_mode_for_size ((word+1) * UNITS_PER_WORD,
-						      GET_MODE_CLASS (mode));
+ if (word == 0)
+   need_mode = mode;
+ else
+   need_mode = smallest_mode_for_size (GET_MODE_SIZE (mode)
+				        + word*UNITS_PER_WORD,
+					GET_MODE_CLASS (mode));

or something similar.


r~