Ticket #147 (new defect)

Opened 5 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.