Le samedi 19 février 2011 à 10:45 +0000, Mark Dickinson a écrit :
> Mark Dickinson <dickinsm@gmail.com> added the comment:
>
> Unassigning. I don't think that r82043 is the *real* culprit here; that bugfix just happened to expose a deficiency in the peepholer; one that's already present in other situations:
>
> >>> dis.dis(lambda: 2*(3*4))
> 1 0 LOAD_CONST 1 (2)
> 3 LOAD_CONST 4 (12)
> 6 BINARY_MULTIPLY
> 7 RETURN_VALUE
> >>> dis.dis(lambda: (2*3)*4)
> 1 0 LOAD_CONST 5 (24)
> 3 RETURN_VALUE
>
> Antoine, does your patch take care of this case too?
It should. Can you test? |