This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] [AArch64] Fix over length lines around aarch64_save_or_restore_fprs
- From: Marcus Shawcroft <marcus dot shawcroft at arm dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 20 Nov 2013 15:52:26 +0000
- Subject: Re: [PATCH] [AArch64] Fix over length lines around aarch64_save_or_restore_fprs
- Authentication-results: sourceware.org; auth=none
- References: <528B9443 dot 6050406 at arm dot com> <528B96C2 dot 50804 at arm dot com>
On 19/11/13 16:50, Kyrill Tkachov wrote:
On 19/11/13 16:39, Marcus Shawcroft wrote:
Committed.
/Marcus
2013-11-19 Marcus Shawcroft <marcus.shawcroft@arm.com>
* config/aarch64/aarch64.c (aarch64_save_or_restore_fprs): Fix over
length lines.
Minor nit... but ENOPATCH ;)
Kyrill
Ooops, sorry.
Attached
/Marcus
diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index cfda95e..4faa7cf 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -1796,7 +1796,8 @@ aarch64_save_or_restore_fprs (int start_offset, int increment,
unsigned regno;
unsigned regno2;
rtx insn;
- rtx (*gen_mem_ref)(enum machine_mode, rtx) = (frame_pointer_needed)? gen_frame_mem : gen_rtx_MEM;
+ rtx (*gen_mem_ref)(enum machine_mode, rtx)
+ = (frame_pointer_needed)? gen_frame_mem : gen_rtx_MEM;
for (regno = V0_REGNUM; regno <= V31_REGNUM; regno++)
@@ -1839,16 +1840,17 @@ aarch64_save_or_restore_fprs (int start_offset, int increment,
( gen_load_pairdf (gen_rtx_REG (DFmode, regno), mem,
gen_rtx_REG (DFmode, regno2), mem2));
- add_reg_note (insn, REG_CFA_RESTORE, gen_rtx_REG (DFmode, regno));
- add_reg_note (insn, REG_CFA_RESTORE, gen_rtx_REG (DFmode, regno2));
+ add_reg_note (insn, REG_CFA_RESTORE,
+ gen_rtx_REG (DFmode, regno));
+ add_reg_note (insn, REG_CFA_RESTORE,
+ gen_rtx_REG (DFmode, regno2));
}
/* The first part of a frame-related parallel insn
is always assumed to be relevant to the frame
calculations; subsequent parts, are only
frame-related if explicitly marked. */
- RTX_FRAME_RELATED_P (XVECEXP (PATTERN (insn), 0,
- 1)) = 1;
+ RTX_FRAME_RELATED_P (XVECEXP (PATTERN (insn), 0, 1)) = 1;
regno = regno2;
start_offset += increment * 2;
}
@@ -1859,7 +1861,8 @@ aarch64_save_or_restore_fprs (int start_offset, int increment,
else
{
insn = emit_move_insn (gen_rtx_REG (DFmode, regno), mem);
- add_reg_note (insn, REG_CFA_RESTORE, gen_rtx_REG (DImode, regno));
+ add_reg_note (insn, REG_CFA_RESTORE,
+ gen_rtx_REG (DImode, regno));
}
start_offset += increment;
}
--
1.7.9.5