From: Simon Marchi <simon.marchi@polymtl.ca>
To: gdb-patches@sourceware.org
Cc: Eli Zaretskii <eliz@gnu.org>,
Jon Turney <jon.turney@dronecode.org.uk>,
Simon Marchi <simon.marchi@polymtl.ca>
Subject: [PATCH 0/7] Add "Windows" OS ABI
Date: Mon, 16 Mar 2020 13:08:38 -0400 [thread overview]
Message-ID: <20200316170845.184386-1-simon.marchi@polymtl.ca> (raw)
This patchset started out as a single patch to have the OS ABI Cygwin
applied to Windows x86-64 binaries, here:
https://sourceware.org/legacy-ml/gdb-patches/2020-03/msg00195.html
with the follow-up here:
https://sourceware.org/pipermail/gdb-patches/2020-March/000022.html
Eli pointed out that it doesn't make sense for binaries compilied with
MinGW to have the Cygwin OS ABI, that there should be separate OS ABIs
for Cygwin and non-Cygwin Windows binaries. This already came up in the
following bug report:
https://sourceware.org/bugzilla/show_bug.cgi?id=21500#add_comment
This patchset does a bit of refactor in that area. Most importantly, it:
- adds a "Windows" OS ABI
- makes GDB recognize the proper OS ABI (Cygwin or Windows) when
loading executables
- makes the builtin long type on Cygwin be 64 bits long
Simon Marchi (7):
gdb: recognize 64 bits Windows executables as Cygwin osabi
gdb: move enum gdb_osabi to osabi.h
gdb: add Windows OS ABI
gdb: rename i386-cygwin-tdep.c to i386-windows-tdep.c
gdb: rename content of i386-windows-tdep.c, cygwin to windows
gdb: select "Cygwin" OS ABI for Cygwin binaries
gdb: define builtin long type to be 64 bits on amd64 Cygwin
gdb/Makefile.in | 4 +-
gdb/amd64-windows-tdep.c | 41 ++++++-
gdb/configure.tgt | 10 +-
gdb/defs.h | 31 ------
gdb/gdbarch.h | 1 +
gdb/gdbarch.sh | 1 +
...i386-cygwin-tdep.c => i386-windows-tdep.c} | 45 +++++---
gdb/osabi.c | 1 +
gdb/osabi.h | 32 ++++++
gdb/windows-tdep.c | 101 ++++++++++++++++++
gdb/windows-tdep.h | 6 ++
11 files changed, 215 insertions(+), 58 deletions(-)
rename gdb/{i386-cygwin-tdep.c => i386-windows-tdep.c} (83%)
--
2.25.1
next reply other threads:[~2020-03-16 17:08 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-16 17:08 Simon Marchi [this message]
2020-03-16 17:08 ` [PATCH 1/7] gdb: recognize 64 bits Windows executables as Cygwin osabi Simon Marchi
2020-03-16 17:08 ` [PATCH 2/7] gdb: move enum gdb_osabi to osabi.h Simon Marchi
2020-03-16 17:08 ` [PATCH 3/7] gdb: add Windows OS ABI Simon Marchi
2020-03-16 17:08 ` [PATCH 4/7] gdb: rename i386-cygwin-tdep.c to i386-windows-tdep.c Simon Marchi
2020-03-16 17:08 ` [PATCH 5/7] gdb: rename content of i386-windows-tdep.c, cygwin to windows Simon Marchi
2020-03-16 17:08 ` [PATCH 6/7] gdb: select "Cygwin" OS ABI for Cygwin binaries Simon Marchi
2020-03-16 18:16 ` Christian Biesinger
2020-03-16 18:18 ` Simon Marchi
2020-03-16 19:03 ` Jon Turney
2020-03-16 21:00 ` Simon Marchi
2020-04-01 19:05 ` Tom Tromey
2020-04-01 19:25 ` Simon Marchi
2020-04-01 21:36 ` Pedro Alves
2020-04-01 21:53 ` Simon Marchi
2020-04-02 13:56 ` Pedro Alves
2020-04-02 14:01 ` Simon Marchi
2020-04-02 14:03 ` Pedro Alves
2020-04-02 14:08 ` Simon Marchi
2020-04-02 14:17 ` Simon Marchi
2020-04-02 13:22 ` Tom Tromey
2020-04-02 14:55 ` [PATCH] gdb: use bfd_get_section_contents to read section contents in, is_linked_with_cygwin_dll (was: Re: [PATCH 6/7] gdb: select "Cygwin" OS ABI for Cygwin binaries) Simon Marchi
2020-04-02 14:57 ` [PATCH] gdb: use bfd_get_section_contents to read section contents in, is_linked_with_cygwin_dll Simon Marchi
2020-04-02 19:01 ` Tom Tromey
2020-04-02 19:42 ` Simon Marchi
2020-04-02 19:45 ` Tom Tromey
2020-04-02 19:47 ` Simon Marchi
2020-03-16 17:08 ` [PATCH 7/7] gdb: define builtin long type to be 64 bits on amd64 Cygwin Simon Marchi
2020-03-16 17:46 ` [PATCH 0/7] Add "Windows" OS ABI Eli Zaretskii
2020-03-16 17:48 ` Simon Marchi
2020-03-16 19:04 ` Jon Turney
2020-04-01 21:42 ` Pedro Alves
2020-04-01 21:56 ` Simon Marchi
2020-04-02 3:06 ` [PATCH] gdb: stop using host-dependent signal numbers in, windows-tdep.c (was: Re: [PATCH 0/7] Add "Windows" OS ABI) Simon Marchi
2020-04-02 14:00 ` [PATCH] gdb: stop using host-dependent signal numbers in, windows-tdep.c Pedro Alves
2020-04-02 14:02 ` Simon Marchi
2020-04-02 15:12 ` Eli Zaretskii
2020-04-08 12:45 ` Jon Turney
2020-04-08 18:16 ` Simon Marchi
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=20200316170845.184386-1-simon.marchi@polymtl.ca \
--to=simon.marchi@polymtl.ca \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=jon.turney@dronecode.org.uk \
/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