executor.add_function needs to add multiple functions simultaneously

Bug #879202 reported by Matt Giuca
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mars
Fix Released
High
Matt Giuca

Bug Description

In the LLVM backend, we actually compile functions when they are added to the environment. If multiple functions are added together, they may depend on one another. The current interactive code folds over all of the functions and inserts them one by one -- this means dependencies may not yet be satisfied.

Change the executor interface so that add_function becomes add_functions. The individual backend can then decide how to deal with accepting a set of functions at a time. Interactive would no longer fold over them, just pass all of them to executor.add_functions.

Related branches

Revision history for this message
Matt Giuca (mgiuca) wrote :

Fixed in trunk r1261.

Changed in mars:
status: In Progress → Fix Committed
Matt Giuca (mgiuca)
Changed in mars:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.