Ticket #147 (new defect)
Opened 4 years ago
BoolBinopNode doesn't reuse temps
| Reported by: | dagss | Owned by: | somebody |
|---|---|---|---|
| Priority: | minor | Milestone: | wishlist |
| Component: | Code Generation | Keywords: | |
| Cc: |
Description
Blocked by #77.
Before, BoolBinopNode? was pretty smart about only using one temp, and reuse it for its children. In the transition phase with old and new temp ExprNodes? mixed as children, this is not possible without serious effort. Once #77 is fixed, one could have a look at putting things back in place so that the extra C assignments aren't needed.
It is a pretty minor issue though, as no INCREF/DECREFs are involved, only the number of declared variables.
It is entirely possible to WONTFIX this ticket, but if so, please go over and clean out the comments left there now.
Note: See
TracTickets for help on using
tickets.
