On 06/27/2013 07:59 AM, Marc Glisse wrote:
I assume I can't call directly c_build_vec_perm_expr on the original
arguments without build_non_dependent_expr?

It looks like c_build_vec_perm_expr is safe to take the original arguments, since it doesn't look deep into the expression. So either way is fine.

By the way, should I rename cp_build_vec_perm_expr as build_x_vec_perm_expr, 
since most of its code is copied from build_x_binary_op and not 
cp_build_binary_op?

Makes sense.  OK with that change.

Jason

Reply via email to