On 11/07/2014 13:11, Roman Gareev wrote:
The struct now contains only a single element such that there seems to be no
need for it anymore. Should we remove it? (We could still use a typedef if
you believe the datatype is too long).
I don't mind removing it. However I think that we should choose the
way of transferring of sese region, which is used for translation of
an isl_ast_node_user to Gimple code. Should we transfer it separately
or restore ivs_params later? What do you think?
Sorry, I currently don't see why sese region is needed later. In case it
is, we may keep it, but this would require to pull more code in the
review. I personally would just remove it to finish this review.
Feel free to commit after applying this change.
Cheers,
Tobias
