From: Eli Zaretskii <eliz@gnu.org>
To: Pierre Muller <pierre.muller@ics-cnrs.unistra.fr>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFC] Fix for Go32-v2 native woes
Date: Tue, 06 Apr 2010 17:46:00 -0000 [thread overview]
Message-ID: <83iq84xyoa.fsf@gnu.org> (raw)
In-Reply-To: <001801cad593$8e70daf0$ab5290d0$%muller@ics-cnrs.unistra.fr>
> From: "Pierre Muller" <pierre.muller@ics-cnrs.unistra.fr>
> Date: Tue, 6 Apr 2010 16:15:05 +0200
>
> I finally managed to find a fix, but it is not straightforward:
> I had to add a new xml file in features/i386 directory
> specific for go32v2 target, that does not read i386-sse.xml.
> I had to adapt the code in i386-tdep.c to support
> missing feature_vector and to set tdesc to
> tdesc_i386_go32v2 instead of tdesc_i386 when go32v2
> osabi was detected.
>
> This allows me to use CVS GDB on DJGPP again.
>
> I checked with a testsuite run on gcc-farm that nothing changed
> for at least that other target (amd64-linux).
>
> Similar fixes might be required for other 'old'
> i386 targets that do not support SSE registers.
>
> Comments welcome as usual!
Thanks. But I really hope that a much more elegant solution could be
found. A general layer of code such as i386-tdep.c should not include
any code that is specific to certain targets.
Why do we always read i386-sse.xml or assume that SSE is supported?
We should not assume any CPU features by default if there are CPUs out
there which we support that don't have them.
next parent reply other threads:[~2010-04-06 17:46 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <002a01cad517$d36eab90$7a4c02b0$%muller@ics-cnrs.unistra.fr>
[not found] ` <001801cad593$8e70daf0$ab5290d0$%muller@ics-cnrs.unistra.fr>
2010-04-06 17:46 ` Eli Zaretskii [this message]
2010-04-06 18:41 ` Mark Kettenis
2010-04-07 3:23 ` Eli Zaretskii
2010-04-07 3:20 ` Eli Zaretskii
2010-04-07 6:19 ` H.J. Lu
2010-04-07 6:44 ` Eli Zaretskii
2010-04-07 14:41 ` Daniel Jacobowitz
2010-04-07 17:34 ` Eli Zaretskii
2010-04-07 17:37 ` H.J. Lu
2010-04-06 17:48 ` Eli Zaretskii
[not found] <1242886820451223367@unknownmsgid>
2010-04-07 6:32 ` [RFC] Fix for " H.J. Lu
2010-04-07 17:35 ` H.J. Lu
2010-04-07 17:53 ` Eli Zaretskii
2010-04-07 18:01 ` H.J. Lu
[not found] <002a01cad517$d36eab90$7a4c02b0$@muller@ics-cnrs.unistra.fr>
2010-04-06 14:15 ` Pierre Muller
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=83iq84xyoa.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--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