Opened 6 years ago

Closed 3 months ago

Last modified 3 months ago

#147 closed defect (fixed)

BoolBinopNode doesn't reuse temps

Reported by: dagss Owned by: scoder
Priority: minor Milestone: 0.21
Component: Code Generation Keywords:
Cc:

Description (last modified by scoder)

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.

Change History (3)

comment:1 Changed 3 months ago by scoder

  • Description modified (diff)
  • Owner changed from somebody to scoder
  • Status changed from new to assigned

comment:2 Changed 3 months ago by scoder

  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from assigned to closed

comment:3 Changed 3 months ago by scoder

  • Milestone changed from wishlist to 0.21
Note: See TracTickets for help on using tickets.