Commit: Fix
Nick Clifton
nickc@redhat.com
Mon Feb 8 14:50:00 GMT 2016
Hi Guys,
I am applying the patch below to fix this compile time warning from
newer versions of gcc:
gas/config/tc-ia64.c: In function 'dot_prologue':
gas/config/tc-ia64.c:4363:2: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]
n = popcount (mask);
gas/config/tc-ia64.c:4361:7: note: ...this 'else' clause, but it is not
else
It turns out that the warning is correct, the programmer probably did
intend for the else clause to guard the calculation. But it is safer
to always compute n, even if the mask is zero, so I have left the code
the way it was, just with the formatting fixed.
Cheers
Nick
gas/ChangeLog
2016-02-08 Nick Clifton <nickc@redhat.com>
* config/tc-ia64.c (dot_prologue): Fix formatting.
diff --git a/gas/config/tc-ia64.c b/gas/config/tc-ia64.c
index 570f746..e00b8e1 100644
--- a/gas/config/tc-ia64.c
+++ b/gas/config/tc-ia64.c
@@ -4360,7 +4360,8 @@ dot_prologue (int dummy ATTRIBUTE_UNUSED)
as_warn (_("Pointless use of zero first operand to .prologue"));
else
mask = e.X_add_number;
- n = popcount (mask);
+
+ n = popcount (mask);
if (sep == ',')
parse_operand_and_eval (&e, 0);
More information about the Binutils
mailing list