espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-24 12:23:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

[AARCH64][GAS] Fix two -Wstack-usage warnings.

Browse Source 
Warning triggerd by gcc 5 with -O0 flag.
error: stack usage might be unbounded [-Werror=stack-usage=]

gas/

2016-06-08  Renlin Li  <renlin.li@arm.com>

	* config/tc-aarch64.c (print_operands): Substitute size.
	(output_operand_error_record): Likewise.
This commit is contained in:
Renlin Li
2016-06-09 10:08:08 +01:00
parent 6b10ba223f
commit 08d3b0cc99
2 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gas/ChangeLog
View File

@ -1,3 +1,8 @@
2016-06-08 Renlin Li <renlin.li@arm.com>
* config/tc-aarch64.c (print_operands): Substitute size.
(output_operand_error_record): Likewise.
2016-06-07 Alan Modra <amodra@gmail.com> 2016-06-07 Alan Modra <amodra@gmail.com>
* config/tc-ppc.c (PPC_APUINFO_ISEL, PPC_APUINFO_PMR, * config/tc-ppc.c (PPC_APUINFO_ISEL, PPC_APUINFO_PMR,

12
gas/config/tc-aarch64.c
View File

@ -4231,8 +4231,7 @@ print_operands (char *buf, const aarch64_opcode *opcode,
for (i = 0; i < AARCH64_MAX_OPND_NUM; ++i) for (i = 0; i < AARCH64_MAX_OPND_NUM; ++i)
{ {
const size_t size = 128; char str[128];
char str[size];
/* We regard the opcode operand info more, however we also look into /* We regard the opcode operand info more, however we also look into
the inst->operands to support the disassembling of the optional the inst->operands to support the disassembling of the optional
@ -4244,7 +4243,7 @@ print_operands (char *buf, const aarch64_opcode *opcode,
break; break;
/* Generate the operand string in STR. */ /* Generate the operand string in STR. */
aarch64_print_operand (str, size, 0, opcode, opnds, i, NULL, NULL); aarch64_print_operand (str, sizeof (str), 0, opcode, opnds, i, NULL, NULL);
/* Delimiter. */ /* Delimiter. */
if (str[0] != '\0') if (str[0] != '\0')
@ -4350,8 +4349,7 @@ output_operand_error_record (const operand_error_record *record, char *str)
size_t len = strlen (get_mnemonic_name (str)); size_t len = strlen (get_mnemonic_name (str));
int i, qlf_idx; int i, qlf_idx;
bfd_boolean result; bfd_boolean result;
const size_t size = 2048; char buf[2048];
char buf[size];
aarch64_inst *inst_base = &inst.base; aarch64_inst *inst_base = &inst.base;
const aarch64_opnd_qualifier_seq_t *qualifiers_list; const aarch64_opnd_qualifier_seq_t *qualifiers_list;
@ -4381,7 +4379,7 @@ output_operand_error_record (const operand_error_record *record, char *str)
/* Print the hint. */ /* Print the hint. */
output_info (_(" did you mean this?")); output_info (_(" did you mean this?"));
snprintf (buf, size, "\t%s", get_mnemonic_name (str)); snprintf (buf, sizeof (buf), "\t%s", get_mnemonic_name (str));
print_operands (buf, opcode, inst_base->operands); print_operands (buf, opcode, inst_base->operands);
output_info (_(" %s"), buf); output_info (_(" %s"), buf);
@ -4402,7 +4400,7 @@ output_operand_error_record (const operand_error_record *record, char *str)
if (i != qlf_idx) if (i != qlf_idx)
{ {
/* Mnemonics name. */ /* Mnemonics name. */
snprintf (buf, size, "\t%s", get_mnemonic_name (str)); snprintf (buf, sizeof (buf), "\t%s", get_mnemonic_name (str));
/* Assign the qualifiers. */ /* Assign the qualifiers. */
assign_qualifier_sequence (inst_base, *qualifiers_list); assign_qualifier_sequence (inst_base, *qualifiers_list);