On 06/14/2018 02:10 PM, David Malcolm wrote: > These are mostly pre-approved, but there are some slightly non-trivial > cases in frv.c and mips.c. They looked pretty trivial to me :-)
> > Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. > > Successfully built "cc1" binaries on all 197 configurations in > contrib/config-list.mk. > > OK for trunk? > > gcc/ChangeLog: > * config/frv/frv-protos.h (frv_ifcvt_modify_insn): Strengthen 3rd > param from rtx to rtx_insn *. > * config/frv/frv.c (frv_ifcvt_add_insn): Likewise for "insn" > param. > (frv_ifcvt_modify_insn): Likwise. > (frv_ifcvt_modify_final): Likwise for local "existing_insn", > adding an as_a <rtx_insn *> cast. Likewise for local "insn". > * config/mips/mips.c (r10k_insert_cache_barriers): Add an > as_a <rtx_insn *> cast to local "unprotected_region" once > it's been established that it's not NULL or pc_rtx. > * config/nds32/nds32-relax-opt.c (nds32_group_insns): Strengthen > param "sethi" from rtx to rtx_insn *. > (nds32_group_float_insns): Likewise for param "insn". > * config/vax/vax-protos.h (vax_output_int_add): Likewise for 1st > param. > (vax_output_int_subtract): Likewise. > * config/vax/vax.c (vax_output_int_add): Likewise for param > "insn". > (vax_output_int_subtract): Likewise. > * emit-rtl.c (set_insn_deleted): Likewise, removing cast. > (emit_pattern_after): Likewise for param "after". > (emit_insn_after): Likewise. > (emit_jump_insn_after): Likewise. > (emit_call_insn_after): Likewise. > (emit_debug_insn_after): Likewise. > (emit_pattern_before): Likewise for param "before". > (emit_insn_before): Likewise. > (emit_jump_insn_before): Likewise. > * final.c (get_insn_template): Likewise for param "insn", removing > a cast. > * output.h (get_insn_template): Likewise for 2nd param. > * rtl.h (emit_insn_before): Likewise. > (emit_jump_insn_before): Likewise. > (emit_debug_insn_before_noloc): Likewise. > (emit_insn_after): Likewise. > (emit_jump_insn_after): Likewise. > (emit_call_insn_after): Likewise. > (emit_debug_insn_after): Likewise. > (set_insn_deleted): Likewise for param. OK. Note the tester doesn't test the vax port. It will test nds32-elf (big and little endian), mips and frv though. jeff