Hi David,
[src/gas/config/tc-v850.c:1663]: (style) Expression is always false because 'else if' condition matches previous condition at line 1655.
Thanks for reporting this bug. I have fixed it by applying the patch below. Cheers Nick gas/ChangeLog 2015-03-05 Nick Clifton <ni...@redhat.com> * config/tc-v850.c (md_parse_option): Fix code to set or clear EF_RH850_DATA_ALIGN8 bit in ELF header, based upon the use of the -m8byte-align and -m4byte-align command line options. diff --git a/gas/config/tc-v850.c b/gas/config/tc-v850.c index fc55106..dec8739 100644 --- a/gas/config/tc-v850.c +++ b/gas/config/tc-v850.c @@ -1653,17 +1653,19 @@ md_parse_option (int c, char *arg) v850_target_format = "elf32-v850-rh850"; } else if (strcmp (arg, "8byte-align") == 0) - v850_data_8 = TRUE; + { + v850_data_8 = TRUE; + v850_e_flags |= EF_RH850_DATA_ALIGN8; + } else if (strcmp (arg, "4byte-align") == 0) - v850_data_8 = FALSE; + { + v850_data_8 = FALSE; + v850_e_flags &= ~ EF_RH850_DATA_ALIGN8; + } else if (strcmp (arg, "soft-float") == 0) soft_float = 1; else if (strcmp (arg, "hard-float") == 0) soft_float = 0; - else if (strcmp (arg, "8byte-align") == 0) - v850_e_flags |= EF_RH850_DATA_ALIGN8; - else if (strcmp (arg, "4byte-align") == 0) - v850_e_flags &= ~ EF_RH850_DATA_ALIGN8; else return 0; _______________________________________________ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils