Steve Fink (via RT) wrote:

> - In imcc.y:main(), the stacktop was being set to an uninitialized
>   value, making stackwalking sometimes *really* slow, sometimes crash,
>   and sometimes work perfectly fine.


Yes, already fixed


> - My bison does not accept actions that do not end in a semicolon, eg
>   { $$ = 0 }


Ditto


> - BRANCH is generated by imcc.l, but is never referred to in imcc.y,


I tossed it.


>   ... This patch adds a 'keyvec' bitmap for this
>   purpose, and removes the VTKEY flag which (I think) is now
>   unnecessary.


Thanks for that, much clearer, simpler, better.

 
> After this patch, all tests pass, but there's a warning from
> t/compiler/6_6.imc because it has a multiply-defined local variable.
> But that's a perl6 problem. It is possible, but highly unlikely, that
> this is a result of my local modifications to perl6 (but they're all
> only in the regex code.)


I have the same warning. P6C is to blame.


> -- at


I'll resent my cumulated fixes in a minute, containing all fixes sent to 
the list.

Thanks for the patch,
leo

Reply via email to