Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Toshihito Kikuchi <k.toshihito@yahoo.de>
To: Pedro Alves <palves@redhat.com>, gdb-patches@sourceware.org
Cc: Paul_Koning@Dell.com, andrew.burgess@embecosm.com, jhb@freebsd.org
Subject: Re: [PATCH] Negative repeat count for 'x' command
Date: Thu, 28 Apr 2016 18:06:00 -0000	[thread overview]
Message-ID: <572250E8.90207@yahoo.de> (raw)
In-Reply-To: <5714DE73.5010104@redhat.com>

Hi Pedro,

I've sent V3 patch today.

> BTW, aren't these tests dependent on the host's charset?  (show host-charset)

Thank you for pointing this out. Yes, then I added 'set host-charset ASCII'
before running string testcases.

> An idea to generalize the x/i tests to all archs would be to
> let go of the asm, and instead write the test function in C.  Then
> you'd first use forward x/i to store a few line's instructions in
> a list/array, and afterwards you'd disassemble backwards, comparing
> with the expected instructions stored in the stored list/array.
> I wonder whether that'd work.

This was great suggestion.  I realized .exp test scripts can support complex
logics more than expected.  I added a new section 'backward disassemble general'
to do that.

A new section "address zero boundary" is also added to test around address zero.
 Since I don't have non-MMU machines, I verified it using by setting
/proc/sys/vm/mmap_min_addr to 0.

Thanks,
Toshihito


      parent reply	other threads:[~2016-04-28 18:06 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1827952218.466587.1453670934999.JavaMail.yahoo.ref@mail.yahoo.com>
2016-01-24 21:31 ` [PATCH] Add a new format letter to dump instructions backward Toshihito Kikuchi
2016-01-25 11:41   ` Pedro Alves
2016-01-25 18:10     ` John Baldwin
2016-01-25 18:22       ` Paul_Koning
2016-01-25 18:27         ` Pedro Alves
2016-01-25 20:11           ` Paul_Koning
2016-01-26  5:39             ` Toshihito Kikuchi
2016-01-27 16:04               ` Andrew Burgess
2016-01-27 20:09                 ` Paul_Koning
2016-01-28 11:53                   ` Pedro Alves
2016-02-07 23:19                     ` [PATCH] Negative repeat count for 'x' command Toshihito Kikuchi
2016-02-17 11:38                       ` Pedro Alves
2016-03-23  0:27                         ` Toshihito Kikuchi
2016-04-11 12:17                       ` Pedro Alves
2016-04-15  0:30                         ` Toshihito Kikuchi
     [not found]                           ` <5714DE73.5010104@redhat.com>
2016-04-28 18:06                             ` Toshihito Kikuchi [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=572250E8.90207@yahoo.de \
    --to=k.toshihito@yahoo.de \
    --cc=Paul_Koning@Dell.com \
    --cc=andrew.burgess@embecosm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jhb@freebsd.org \
    --cc=palves@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox