Re: [PATCH 2/5] Don't modify recog_op_alt after preprocess_constraints

[Jeff Law]

On 05/31/14 03:09, Richard Sandiford wrote:

Since the aim of this series is to cache the result of preprocess_constraints,
we need to make sure that passes don't modify the information afterwards.
This patch deals with the places that did.  Patch 4 will make the information
properly const.

Thanks,
Richard


gcc/
        * recog.h (alternative_class): New function.
        (which_op_alt): Return a const recog_op_alt.
        * reg-stack.c (check_asm_stack_operands): Update type accordingly.
        (subst_asm_stack_regs): Likewise.
        * config/arm/arm.c (note_invalid_constants): Likewise.
        * regcprop.c (copyprop_hardreg_forward_1): Likewise.  Don't modify
        the operand_alternative; use alternative class instead.
        * sel-sched.c (get_reg_class): Likewise.
        * regrename.c (build_def_use): Likewise.
        (hide_operands, restore_operands, record_out_operands): Update type
        accordingly.

OK.
Jeff