yggdrasil/binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use value_contents instead of value_contents_writeable

Browse Source 
Both aarch64_push_dummy_call and bfin_push_dummy_call only use args[i]
contents but then never write to them, so that we can use
value_contents instead.

gdb:

2015-11-16  Yao Qi  <yao.qi@linaro.org>

	* aarch64-tdep.c (aarch64_push_dummy_call): Call value_contents instead
	of value_contents_writeable.
	* bfin-tdep.c (bfin_push_dummy_call): Likewise.
This commit is contained in:
Yao Qi 2015-11-16 14:47:50 +00:00
parent ef9bd0b8d7
commit 0d1993c072
3 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gdb/ChangeLog
View File

@ -1,3 +1,9 @@
2015-11-16 Yao Qi <yao.qi@linaro.org>
* aarch64-tdep.c (aarch64_push_dummy_call): Call value_contents
instead of value_contents_writeable.
* bfin-tdep.c (bfin_push_dummy_call): Likewise.
2015-11-16 Yao Qi <yao.qi@linaro.org>
* arm-tdep.c (arm_push_dummy_call): New array buf. Store regval

2
gdb/aarch64-tdep.c
View File

@ -1300,7 +1300,7 @@ aarch64_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
check_typedef (value_type (field));
pass_in_v_or_stack (gdbarch, regcache, &info, field_type,
value_contents_writeable (field));
value_contents (field));
}
}
else

2
gdb/bfin-tdep.c
View File

@ -531,7 +531,7 @@ bfin_push_dummy_call (struct gdbarch *gdbarch,
int container_len = (TYPE_LENGTH (value_type) + 3) & ~3;
sp -= container_len;
write_memory (sp, value_contents_writeable (args[i]), container_len);
write_memory (sp, value_contents (args[i]), container_len);
}
/* Initialize R0, R1, and R2 to the first 3 words of parameters. */