[PATCH] PR70674: S/390: Add memory barrier to stack pointer restore from fpr.

Andreas Krebbel krebbel@linux.vnet.ibm.com
Tue Apr 19 16:14:00 GMT 2016


On 04/19/2016 12:54 PM, Jakub Jelinek wrote:
> Can you please:
> 	rtx fpr = gen_rtx_REG (DImode, cfun_gpr_save_slot (i));
> 	if (i == STACK_POINTER_REGNUM)
> 	  insn = emit_insn (gen_stack_restore_from_fpr (fpr));
> 	else
> 	  insn = emit_move_insn (gen_rtx_REG (DImode, i), fpr);
> That way IMHO it is more nicely formatted, you avoid the ugly (
> at the end of line, it uses fewer lines anyway and additionally
> you can make it clear what the gen_rtx_REG (DImode, cfun_gpr_save_slot (i))
> means by giving it a name.  Of course, choose whatever other var
> name you prefer to describe what it is.

Right, that's better. I'll change the patch and commit it tomorrow. Thanks!

-Andreas-



More information about the Gcc-patches mailing list