[PATCH v3 2/2] gdb/s390-linux: Step over MVCLE+JO (and similiar) as a unit.
Marcin Kościelnicki
koriakin@0x04.net
Tue Nov 3 16:44:00 GMT 2015
On 03/11/15 17:13, Ulrich Weigand wrote:
> Marcin KoÅcielnicki <koriakin at 0x04 dot net> wrote:
>
>> gdb/ChangeLog:
>>
>> PR/18376
>> * gdb/s390-linux-tdep.c (s390_is_partial_instruction): New function.
>> (s390_software_single_step): New function.
>> (s390_displaced_step_hw_singlestep): New function.
>> (s390_gdbarch_init): Fill gdbarch slots with the above.
>
>> + /* Special handling only if recording. */
>> + if (!record_full_is_used())
>> + return 0;
>> +
>> + /* First, match a partial instruction. */
>> + if (!s390_is_partial_instruction(gdbarch, loc, &len))
>> + return 0;
>
> Coding style issue: space before ( for function calls.
>
> Patch is OK with that change.
>
> Thanks,
> Ulrich
>
Thanks for the review. Should I push these two patches already, or wait
for review on the xmalloc one first?
[https://sourceware.org/ml/gdb-patches/2015-11/msg00021.html]
More information about the Gdb-patches
mailing list