Matt Fowles <[EMAIL PROTECTED]> wrote:

> This patch adds simple constant propagation to imcc.

First thanks, seconds I modified and applied it - but third, its
disabled and doesn't work and forth - it needs a lot of tests to really
get activated.

I did add some comments, and an example, why it will not work.
I think its better, to have the midifed patch applied and leave this as
some piece of documentation in the code.

> I am a little confused as to why I had to do

> for(i = ins2->opsize - 2; i >= 0; i--)

> rather than

> for(i = ins2->opsize - 1; i >= 0; i--)

Parrots opsize contains the opcode too. So opsize-1 is the arg count.

> Comments welcome,
> Matt

leo

Reply via email to