tbsaunde+...@tbsaunde.org writes: > @@ -265,18 +261,16 @@ store_ops_ok (const_rtx x, int *regs_set) > /* Returns a list of registers mentioned in X. > FIXME: A regset would be prettier and less expensive. */ > > -static rtx_expr_list * > -extract_mentioned_regs (rtx x) > +static void > +extract_mentioned_regs (rtx x, vec<rtx> *mentioned_regs) > { > - rtx_expr_list *mentioned_regs = NULL; > subrtx_var_iterator::array_type array; > FOR_EACH_SUBRTX_VAR (iter, array, x, NONCONST) > { > rtx x = *iter; > if (REG_P (x)) > - mentioned_regs = alloc_EXPR_LIST (0, x, mentioned_regs); > + mentioned_regs->safe_push (x); > } > - return mentioned_regs; > }
The function comment needs to be updated. Thanks, Richard