> > + if (is_gimple_min_invariant (op)) > > + return true; > > + if (SSA_NAME_IS_DEFAULT_DEF (op) > > + || !flow_bb_inside_loop_p (loop, gimple_bb (SSA_NAME_DEF_STMT > (op)))) > > + return true; > > + return gimple_uid (SSA_NAME_DEF_STMT (op)) & 1; > > +} > > +
Does gimple_uid ever return something useful for us here? In tree-ssa-loop-ch it is being populated before and then used but I don't think we populate it properly? So my question would be, isn't is_gimple_constant and flow_bb_inside_loop_p sufficient for our purpose? Regards Robin