OpenE2K/binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

AArch64: Replace C initializers with memset

Browse Source 
Clang doesn't accept {0} as a valid C struct initializer under their implementation
of -Wmissing-field-initializers.  This makes using C initializers a bit tricky.

Instead I'm changing the code to use memset instead, which at least GCC inlines and
generates the same code for.  This also seems to be the idiom used in binutils for
most targets.

opcodes/

	* aarch64-opc.c (verify_constraints): Use memset instead of {0}.
This commit is contained in:
Tamar Christina 2018-10-08 13:33:42 +01:00
parent 80f3ac5d61
commit 780f601cf3
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
opcodes/ChangeLog
View File

@ -1,3 +1,7 @@
2018-10-08 Tamar Christina <tamar.christina@arm.com>
* aarch64-opc.c (verify_constraints): Use memset instead of {0}.
2018-10-05 H.J. Lu <hongjiu.lu@intel.com>
* i386-dis.c (rm_table): Add enclv.

4
opcodes/aarch64-opc.c
View File

@ -4671,7 +4671,9 @@ verify_constraints (const struct aarch64_inst *inst,
/* Next check for usage of the predicate register. */
aarch64_opnd_info blk_dest = insn_sequence->instr->operands[0];
aarch64_opnd_info blk_pred = {0}, inst_pred = {0};
aarch64_opnd_info blk_pred, inst_pred;
memset (&blk_pred, 0, sizeof (aarch64_opnd_info));
memset (&inst_pred, 0, sizeof (aarch64_opnd_info));
bfd_boolean predicated = FALSE;
assert (blk_dest.type == AARCH64_OPND_SVE_Zd);