Hi John,
yes I think it might be interesting if you could add the following lines
inside m2expr_BuildBinarySetDo:
void
m2expr_BuildBinarySetDo (location_t location, tree settype, tree op1, tree op2,
tree op3,
void (*binop)(location_t, tree, tree, tree, tree,
int),
int is_op1lvalue, int is_op2lvalue, int is_op3lvalue,
tree nBits,
tree unbounded,
tree varproc, tree leftproc, tree rightproc)
{
tree size = m2expr_GetSizeOf (location, settype);
int is_const = FALSE;
int is_left = FALSE;
m2assert_AssertLocation (location);
ASSERT_BOOL (is_op1lvalue);
ASSERT_BOOL (is_op2lvalue);
ASSERT_BOOL (is_op3lvalue);
pe (op1); /* six new lines. */
pe (op2);
pe (op3);
debug_tree (op1);
debug_tree (op2);
debug_tree (op3);