public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Zac Walker <zacwalker@microsoft.com>
To: Christophe Lyon <christophe.lyon@linaro.org>,
	Evgeny Karpov <Evgeny.Karpov@microsoft.com>
Cc: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>,
	"richard.sandiford@arm.com" <richard.sandiford@arm.com>,
	"Richard Earnshaw (lists)" <Richard.Earnshaw@arm.com>,
	Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org>,
	Radek Barton <radek.barton@microsoft.com>
Subject: RE: [EXTERNAL] Re: [PATCH v3 00/12] Add aarch64-w64-mingw32 target
Date: Tue, 7 May 2024 16:31:56 +0000	[thread overview]
Message-ID: <PA6PR83MB05971D7BB40FA28630A91E29BFE42@PA6PR83MB0597.EURPRD83.prod.outlook.com> (raw)
In-Reply-To: <CAPS5khYn6ve1tQTJPA4CRCCyKVpMWHE6Z4pAUrJiyr7mjdj7RA@mail.gmail.com>

Cool - congratulations everyone!!
Thanks for getting it completed. Fantastic effort from you all.

Zac

-----Original Message-----
From: Christophe Lyon <christophe.lyon@linaro.org> 
Sent: Tuesday, May 7, 2024 6:06 PM
To: Evgeny Karpov <Evgeny.Karpov@microsoft.com>
Cc: gcc-patches@gcc.gnu.org; richard.sandiford@arm.com; Richard Earnshaw (lists) <Richard.Earnshaw@arm.com>; Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org>; Radek Barton <radek.barton@microsoft.com>; Zac Walker <zacwalker@microsoft.com>
Subject: [EXTERNAL] Re: [PATCH v3 00/12] Add aarch64-w64-mingw32 target

Hi,

I've just pushed this patch series, congratulations!

Thanks,

Christophe


On Thu, 11 Apr 2024 at 15:40, Evgeny Karpov <Evgeny.Karpov@microsoft.com> wrote:
>
> Hello,
>
> Thank you for reviewing v2!
> v3 addresses all comments on v2.
>
> v3 Changes:
> - Exclude the aarch64_calling_abi declaration from the patch series.
> - Refactor x18 adjustment for MS ABI.
> - Remove unnecessary headers.
> - Add an extra comment to explain empty definitions.
> - Use gcc_unreachable for definitions that are needed for compilation, 
> but not used by the aarch64-w64-mingw32 target.
> - Retain old index entries.
> - Rebase from 11th April 2024
>
> Regards,
> Evgeny
>
>
> Zac Walker (12):
>   Introduce aarch64-w64-mingw32 target
>   aarch64: Mark x18 register as a fixed register for MS ABI
>   aarch64: Add aarch64-w64-mingw32 COFF
>   Reuse MinGW from i386 for AArch64
>   Rename section and encoding functions from i386 which will be used in
>     aarch64
>   Exclude i386 functionality from aarch64 build
>   aarch64: Add Cygwin and MinGW environments for AArch64
>   aarch64: Add SEH to machine_function
>   Rename "x86 Windows Options" to "Cygwin and MinGW Options"
>   aarch64: Build and add objects for Cygwin and MinGW for AArch64
>   aarch64: Add aarch64-w64-mingw32 target to libatomic
>   Add aarch64-w64-mingw32 target to libgcc
>
>  fixincludes/mkfixinc.sh                       |   3 +-
>  gcc/config.gcc                                |  47 +++--
>  gcc/config/aarch64/aarch64-abi-ms.h           |  34 ++++
>  gcc/config/aarch64/aarch64-coff.h             |  91 +++++++++
>  gcc/config/aarch64/aarch64-protos.h           |   5 +
>  gcc/config/aarch64/aarch64.h                  |  13 +-
>  gcc/config/aarch64/cygming.h                  | 172 ++++++++++++++++++
>  gcc/config/i386/cygming.h                     |  18 +-
>  gcc/config/i386/cygming.opt.urls              |  30 ---
>  gcc/config/i386/i386-protos.h                 |  12 +-
>  gcc/config/i386/mingw-w64.opt.urls            |   2 +-
>  gcc/config/lynx.opt.urls                      |   2 +-
>  gcc/config/{i386 => mingw}/cygming.opt        |   0
>  gcc/config/mingw/cygming.opt.urls             |  30 +++
>  gcc/config/{i386 => mingw}/cygwin-d.cc        |   0
>  gcc/config/{i386 => mingw}/mingw-stdint.h     |   9 +-
>  gcc/config/{i386 => mingw}/mingw.opt          |   0
>  gcc/config/{i386 => mingw}/mingw.opt.urls     |   2 +-
>  gcc/config/{i386 => mingw}/mingw32.h          |   4 +-
>  gcc/config/{i386 => mingw}/msformat-c.cc      |   0
>  gcc/config/{i386 => mingw}/t-cygming          |  23 ++-
>  gcc/config/{i386 => mingw}/winnt-cxx.cc       |   0
>  gcc/config/{i386 => mingw}/winnt-d.cc         |   0
>  gcc/config/{i386 => mingw}/winnt-stubs.cc     |   0
>  gcc/config/{i386 => mingw}/winnt.cc           |  30 +--
>  gcc/doc/invoke.texi                           |  10 +
>  gcc/varasm.cc                                 |   2 +-
>  libatomic/configure.tgt                       |   2 +-
>  libgcc/config.host                            |  23 ++-
>  libgcc/config/aarch64/t-no-eh                 |   2 +
>  libgcc/config/{i386 => mingw}/t-gthr-win32    |   0
>  libgcc/config/{i386 => mingw}/t-mingw-pthread |   0
>  32 files changed, 473 insertions(+), 93 deletions(-)  create mode 
> 100644 gcc/config/aarch64/aarch64-abi-ms.h
>  create mode 100644 gcc/config/aarch64/aarch64-coff.h  create mode 
> 100644 gcc/config/aarch64/cygming.h  delete mode 100644 
> gcc/config/i386/cygming.opt.urls  rename gcc/config/{i386 => 
> mingw}/cygming.opt (100%)  create mode 100644 
> gcc/config/mingw/cygming.opt.urls  rename gcc/config/{i386 => 
> mingw}/cygwin-d.cc (100%)  rename gcc/config/{i386 => 
> mingw}/mingw-stdint.h (86%)  rename gcc/config/{i386 => 
> mingw}/mingw.opt (100%)  rename gcc/config/{i386 => 
> mingw}/mingw.opt.urls (86%)  rename gcc/config/{i386 => 
> mingw}/mingw32.h (99%)  rename gcc/config/{i386 => 
> mingw}/msformat-c.cc (100%)  rename gcc/config/{i386 => 
> mingw}/t-cygming (73%)  rename gcc/config/{i386 => mingw}/winnt-cxx.cc 
> (100%)  rename gcc/config/{i386 => mingw}/winnt-d.cc (100%)  rename 
> gcc/config/{i386 => mingw}/winnt-stubs.cc (100%)  rename 
> gcc/config/{i386 => mingw}/winnt.cc (97%)  create mode 100644 
> libgcc/config/aarch64/t-no-eh  rename libgcc/config/{i386 => 
> mingw}/t-gthr-win32 (100%)  rename libgcc/config/{i386 => 
> mingw}/t-mingw-pthread (100%)
>
> --
> 2.25.1
>

      reply	other threads:[~2024-05-07 16:31 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-11 13:38 Evgeny Karpov
2024-04-11 13:45 ` [PATCH v3 01/12] Introduce " Evgeny Karpov
2024-04-11 13:48 ` [PATCH v3 02/12] aarch64: Mark x18 register as a fixed register for MS ABI Evgeny Karpov
2024-04-11 13:50 ` [PATCH v3 03/12] aarch64: Add aarch64-w64-mingw32 COFF Evgeny Karpov
2024-04-11 13:51 ` [PATCH v3 04/12] Reuse MinGW from i386 for AArch64 Evgeny Karpov
2024-04-11 13:53 ` [PATCH v3 05/12] Rename section and encoding functions from i386 which will be used in aarch64 Evgeny Karpov
2024-04-11 13:54 ` [PATCH v3 06/12] Exclude i386 functionality from aarch64 build Evgeny Karpov
2024-04-11 13:56 ` [PATCH v3 07/12] aarch64: Add Cygwin and MinGW environments for AArch64 Evgeny Karpov
2024-04-11 13:58 ` [PATCH v3 08/12] aarch64: Add SEH to machine_function Evgeny Karpov
2024-04-11 13:59 ` [PATCH v3 09/12] Rename "x86 Windows Options" to "Cygwin and MinGW Options" Evgeny Karpov
2024-04-11 14:00 ` [PATCH v3 10/12] aarch64: Build and add objects for Cygwin and MinGW for AArch64 Evgeny Karpov
2024-04-11 14:01 ` [PATCH v3 11/12] aarch64: Add aarch64-w64-mingw32 target to libatomic Evgeny Karpov
2024-04-11 14:03 ` [PATCH v3 12/12] Add aarch64-w64-mingw32 target to libgcc Evgeny Karpov
2024-05-07 16:05 ` [PATCH v3 00/12] Add aarch64-w64-mingw32 target Christophe Lyon
2024-05-07 16:31   ` Zac Walker [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=PA6PR83MB05971D7BB40FA28630A91E29BFE42@PA6PR83MB0597.EURPRD83.prod.outlook.com \
    --to=zacwalker@microsoft.com \
    --cc=Evgeny.Karpov@microsoft.com \
    --cc=Richard.Earnshaw@arm.com \
    --cc=christophe.lyon@linaro.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=maxim.kuvyrkov@linaro.org \
    --cc=radek.barton@microsoft.com \
    --cc=richard.sandiford@arm.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).