Re: [Valgrind-developers] Typo in vg_skin.h

[Nicholas N. <nj...@ca...>]

On 2 Oct 2002, Jeremy Fitzhardinge wrote:

> Surely it should be:
>
> -#define ALL_RREGS_LIVE      (1 << (VG_MAX_REALREGS-1))  /* 0011...11b */
> +#define ALL_RREGS_LIVE      ((1 << VG_MAX_REALREGS)-1)  /* 0011...11b */

Absolutely.  I was wondering how anything worked at all with such an
egregious error... turns out this value is only being used to initialise
the reg liveness info, and is always overwritten by the liveness analysis.
I just fixed it.

Thanks.

N