Message128844
| Author |
pitrou |
| Recipients |
jdharper, mark.dickinson, pitrou, r.david.murray, rhettinger |
| Date |
2011年02月19日.11:04:57 |
| SpamBayes Score |
0.0019993233 |
| Marked as misclassified |
No |
| Message-id |
<1298113493.3793.0.camel@localhost.localdomain> |
| In-reply-to |
<1298112337.11.0.0847516563791.issue11244@psf.upfronthosting.co.za> |
| Content |
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? |
|