Re: [PATCH] Fix alter_output_for_subst_insn (PR other/77421)

2016-09-05 Thread Richard Biener
On Fri, 2 Sep 2016, Jakub Jelinek wrote: > Hi! > > The two ports that use define_subst (ix86 and visium) don't do anything in > this function, other than returning early - return insn_out, so all I could > do is look at the intent. > The *insn_out == '*' || *insn_out != '@' got flagged by some to

[PATCH] Fix alter_output_for_subst_insn (PR other/77421)

2016-09-02 Thread Jakub Jelinek
Hi! The two ports that use define_subst (ix86 and visium) don't do anything in this function, other than returning early - return insn_out, so all I could do is look at the intent. The *insn_out == '*' || *insn_out != '@' got flagged by some tool, the "*insn_out == '*' || " part is unnecessary, si