Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "H.J. Lu" <hjl.tools@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Pierre Muller <pierre.muller@ics-cnrs.unistra.fr>,
	mark.kettenis@xs4all.nl,  	gdb-patches@sourceware.org
Subject: Re: PATCH: Support i386 without SSE
Date: Thu, 08 Apr 2010 22:29:00 -0000	[thread overview]
Message-ID: <v2r6dc9ffc81004081528tb1c11719i4c67d7e82fc0e5af@mail.gmail.com> (raw)
In-Reply-To: <838w8x1va8.fsf@gnu.org>

On Thu, Apr 8, 2010 at 2:36 PM, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: "Pierre Muller" <pierre.muller@ics-cnrs.unistra.fr>
>> Cc: <gdb-patches@sourceware.org>
>> Date: Thu, 8 Apr 2010 22:33:12 +0200
>>
>> > > This patch supports i386 without SSE.  Any comments/suggestions?
>> >
>> > Eli, Pierre, can you test wether this fixes the issues on Go32?
>>
>>  I tested the patch and it does
>> indeed fix the two problems that I
>> reported.
>
> Thanks for testing.

I will check in my patch.

>>   Nevertheless, there are still strange things:
>>   if you use 'maint print register-groups'
>> you will find out that
>> the SSE xmmX registers are still present as registers 32 to 40
>> and the ymmX as 41 to 48, but with empty names and a strange int0_t type.

I think those are expected as they are reserved internally
and not visible to user.

> H.J., could this be fixed?  (I assume it's also part of one of your
> latest patches; apologies if that's not true.)
>

I will post a patch to remove "general" from missing registers.
As for void * registers, we always print

(top-gdb) maint print register-groups
 Name         Nr  Rel Offset    Size  Type            Groups
 eax           0    0      0       4 int             general,all,save,restore
 ecx           1    1      4       4 int             general,all,save,restore
 edx           2    2      8       4 int             general,all,save,restore
 ebx           3    3     12       4 int             general,all,save,restore
 esp           4    4     16       4 *1              general,all,save,restore
 ebp           5    5     20       4 *1              general,all,save,restore
 esi           6    6     24       4 int             general,all,save,restore
 edi           7    7     28       4 int             general,all,save,restore
 eip           8    8     32       4 *1              general,all,save,restore

-- 
H.J.


  reply	other threads:[~2010-04-08 22:29 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-07 20:05 H.J. Lu
2010-04-07 20:12 ` H.J. Lu
2010-04-07 20:31   ` Eli Zaretskii
2010-04-08 19:08 ` Mark Kettenis
2010-04-08 20:33   ` Pierre Muller
     [not found]   ` <000c01cad75a$b751a490$25f4edb0$%muller@ics-cnrs.unistra.fr>
2010-04-08 21:37     ` Eli Zaretskii
2010-04-08 22:29       ` H.J. Lu [this message]
2010-04-09  7:41         ` Eli Zaretskii
2010-04-09 13:16           ` H.J. Lu
2010-04-09 15:08             ` Eli Zaretskii

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=v2r6dc9ffc81004081528tb1c11719i4c67d7e82fc0e5af@mail.gmail.com \
    --to=hjl.tools@gmail.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=mark.kettenis@xs4all.nl \
    --cc=pierre.muller@ics-cnrs.unistra.fr \
    /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