This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [COMMITTED PATCH 1/3] ARM: extract_arm_insn function need to read instrs correctly in be8 case
- From: Victor Kamensky <victor dot kamensky at linaro dot org>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Mon, 3 Nov 2014 09:11:40 -0800
- Subject: Re: [COMMITTED PATCH 1/3] ARM: extract_arm_insn function need to read instrs correctly in be8 case
- Authentication-results: sourceware.org; auth=none
- References: <1414964484-6270-1-git-send-email-victor dot kamensky at linaro dot org> <87bnoosckj dot fsf at codesourcery dot com>
On 3 November 2014 04:17, Yao Qi <yao@codesourcery.com> wrote:
> Victor Kamensky <victor.kamensky@linaro.org> writes:
>
>> 2014-11-02 Victor Kamensky <victor.kamensky@linaro.org>
>>
>> * arm-tdep.c (extract_arm_insn): Use
>> gdbarch_byte_order_for_code to read arm instruction.
>
> Victor,
> The changelog entry should be added to the corresponding ChangeLog files
> on commit. They are missing from your commits. I add them back.
Sorry, I did not realize that. Will do in the future.
I assume those ChangeLog pieces should not be
removed from commit messages. I.e they will present
in both places corresponding ChangeLog file and
commit messages. Is it correct?
Thanks,
Victor
> --
> Yao (éå)
>
> Subject: [PATCH] Add missing changelog entries
>
> Some changelog entries are missing in previous commits. Add them.
>
> diff --git a/gdb/ChangeLog b/gdb/ChangeLog
> index f09715c..928efef 100644
> --- a/gdb/ChangeLog
> +++ b/gdb/ChangeLog
> @@ -1,3 +1,20 @@
> +2014-11-02 Victor Kamensky <victor.kamensky@linaro.org>
> +
> + * dwarf2loc.c (read_pieced_value): Do big endian
> + processing only if gdb_regnum is not -1.
> + (write_pieced_value): Ditto.
> +
> +2014-11-02 Victor Kamensky <victor.kamensky@linaro.org>
> +
> + * arm-linux-tdep.c (arm_linux_init_abi): Use
> + info.byte_order_for_code to choose endianity of breakpoint
> + instructions snippets.
> +
> +2014-11-02 Victor Kamensky <victor.kamensky@linaro.org>
> +
> + * arm-tdep.c (extract_arm_insn): Use
> + gdbarch_byte_order_for_code to read arm instruction.
> +
> 2014-11-02 Doug Evans <xdje42@gmail.com>
>
> * mdebugread.c (parse_procedure): Delete unnecessary forward decl.