This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[committed] MIPS/GAS: Correct call formatting in `can_swap_branch_p'
- From: "Maciej W. Rozycki" <macro at imgtec dot com>
- To: <binutils at sourceware dot org>
- Date: Sat, 9 Apr 2016 13:32:32 +0100
- Subject: [committed] MIPS/GAS: Correct call formatting in `can_swap_branch_p'
- Authentication-results: sourceware.org; auth=none
gas/
* config/tc-mips.c (can_swap_branch_p): Correct call formatting.
---
I have committed this change.
Maciej
binutils-mips-gas-swap-branch-format.diff
Index: binutils/gas/config/tc-mips.c
===================================================================
--- binutils.orig/gas/config/tc-mips.c 2016-04-06 02:15:49.037171207 +0100
+++ binutils/gas/config/tc-mips.c 2016-04-06 02:17:31.314345337 +0100
@@ -6733,11 +6733,11 @@ can_swap_branch_p (struct mips_cl_insn *
/* Parameter must be 16 bit. */
&& (*reloc_type == BFD_RELOC_16_PCREL_S2)
/* Branch to same segment. */
- && (S_GET_SEGMENT(address_expr->X_add_symbol) == now_seg)
+ && (S_GET_SEGMENT (address_expr->X_add_symbol) == now_seg)
/* Branch to same code fragment. */
- && (symbol_get_frag(address_expr->X_add_symbol) == frag_now)
+ && (symbol_get_frag (address_expr->X_add_symbol) == frag_now)
/* Can only calculate branch offset if value is known. */
- && symbol_constant_p(address_expr->X_add_symbol)
+ && symbol_constant_p (address_expr->X_add_symbol)
/* Check if branch is really conditional. */
&& !((ip->insn_opcode & 0xffff0000) == 0x10000000 /* beq $0,$0 */
|| (ip->insn_opcode & 0xffff0000) == 0x04010000 /* bgez $0 */
@@ -6746,7 +6746,7 @@ can_swap_branch_p (struct mips_cl_insn *
int distance;
/* Check if loop is shorter than 6 instructions including
branch and delay slot. */
- distance = frag_now_fix() - S_GET_VALUE(address_expr->X_add_symbol);
+ distance = frag_now_fix () - S_GET_VALUE (address_expr->X_add_symbol);
if (distance <= 20)
{
int i;
@@ -6758,7 +6758,7 @@ can_swap_branch_p (struct mips_cl_insn *
for (i = 0; i < (distance / 4); i++)
{
if ((history[i].cleared_p)
- || delayed_branch_p(&history[i]))
+ || delayed_branch_p (&history[i]))
{
rv = TRUE;
break;