Stefan Behnel, 06.12.2010 11:35: > I think it would be best to generate the straight assignment code from > within of the YieldExprNode: > > closure->place_for_int_temp[0] = __pyx_t_2 > closure->place_for_int_temp[1] = __pyx_t_5 > closure->place_for_object_temp[0] = __pyx_t_9 > return result > resume: > __pyx_t_2 = closure->place_for_int_temp[0] > ... > > For starters, I wouldn't mind if the closure fields were all "size_t" and > the assignments just did an ugly cast on the RHS. Might be easier than > properly allocating separate arrays of correctly sized fields.
... although, of course, not every C value fits into size_t, either. There may be structs being passed around by value, for example. Tricky... Hmm, maybe it's really ok to start with temps in the closure for now, just like we accepted that closures were huge until they were cut down to what was actually needed. I think the feature is more important than the optimisations around it. Stefan _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
