Re: [PATCH] d: Add darwin support for D language front-end

2020-12-06 Thread ciel via Gcc-patches
Iain-san,

Thank you for finally working on GDC Darwin support.

T. Yamada

2020年11月30日(月) 7:28 :
>
> Send Gcc-patches mailing list submissions to
> gcc-patches@gcc.gnu.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
> https://gcc.gnu.org/mailman/listinfo/gcc-patches
> or, via email, send a message with subject or body 'help' to
> gcc-patches-requ...@gcc.gnu.org
>
> You can reach the person managing the list at
> gcc-patches-ow...@gcc.gnu.org
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of Gcc-patches digest..."
> Today's Topics:
>
>1. Re: [PATCH] configure: Support building D front-end on
>   *-*-darwin* (Iain Buclaw)
>2. Re: [PATCH] d: Add darwin support for D language front-end
>   (Iain Buclaw)
>3. Fix freeing of thunk_info (Jan Hubicka)
>4. Re: [PATCH] handle conditionals in -Wstringop-overflow et al.
>   (PR 92936) (Martin Sebor)
>
>
>
> -- Forwarded message --
> From: Iain Buclaw 
> To: Iain Sandoe 
> Cc: gcc-patches@gcc.gnu.org
> Bcc:
> Date: Sun, 29 Nov 2020 22:13:58 +0100
> Subject: Re: [PATCH] configure: Support building D front-end on *-*-darwin*
> Excerpts from Iain Sandoe's message of November 29, 2020 10:35 am:
> > Hi Iain
> >
> > Iain Buclaw  wrote:
> >
> >> The bootstrap has been succeeding for some time now, there's no need to
> >> set it as an unsupported language.
> >>
> >> OK for mainline?
> >
> > At present, this breaks bootstrap on 32 bit Darwin hosts.
> >
> > Once that’s resolved, then it seems a reasonable to make sure that the D
> > FE is built and tested (even though there is still work to do on the library
> > support).
> >
> > So, OK once all the currently tested Darwin hosts bootstrap with D enabled.
> >
>
> As confirmed off list, powerpc and i686 darwin9 bootstrapped OK, so I've
> committed it as r11-5521.
>
> Iain.
>
>
>
>
> -- Forwarded message --
> From: Iain Buclaw 
> To: Iain Sandoe 
> Cc: gcc-patches@gcc.gnu.org
> Bcc:
> Date: Sun, 29 Nov 2020 22:21:26 +0100
> Subject: Re: [PATCH] d: Add darwin support for D language front-end
> Excerpts from Iain Sandoe's message of November 29, 2020 1:49 pm:
> > Hi Iain
> >
> > Iain Buclaw  wrote:
> >
> >> This patch adds necessary predefined version symbols and support for
> >> moduleinfo sections for darwin to allow testing libphobos support.
> >>
> >> OK for mainline?
> >
> > As we discussed off-list, this sets ABI (the section names are visible and
> > part
> > of the contract with the runtime).
> >
> > It’s highly desirable that we can interoperate with D objects built by
> > other impls.
> > so the section naming convention should be agreed.
> >
> > As far as I understand things, the names you have here agree with those used
> > by “upstream” (but current the LLVM D variant uses something different).
> >
> > The Darwin parts are OK, with a note to (re-)check on the section names
> > before
> > the first release*.
> >
> > thanks
> > Iain
> >
> > * perhaps it might be worth pinging the relevant upstream and LLV folks long
> > before that - since the release cycles there mean that there will likely be
> > another
> > one before GCC11 ships.
> >
>
> I've sent Martin a message, so hoping for a response within the week
> which name he'd prefer I go.  I suspect it'd be better to align with
> LLVM instead of the reference DMD compiler.  As the compiler/run-time
> integration of modules uses the same machinery, and we both want to
> support loading shared DSOs (DMD is static only on OSX).
>
> Adjusted the patch with some copy/paste fixes (missing tabs, Glibc gets
> a mention).  Committed the following as r11-5522.
>
> Regards,
> Iain
>
> ---
> gcc/ChangeLog:
>
> * config.gcc (*-*-darwin*): Set d_target_objs and target_has_targetdm.
> * config/elfos.h (TARGET_D_MINFO_SECTION): New macro.
> (TARGET_D_MINFO_START_NAME): New macro.
> (TARGET_D_MINFO_END_NAME): New macro.
> * config/t-darwin: Add darwin-d.o.
> * doc/tm.texi: Regenerate.
> * doc/tm.texi.in (D language and ABI): Add @hook for
> TARGET_D_MINFO_SECTION, TARGET_D_MINFO_START_NAME, and
> TARGET_D_MINFO_END_NAME.
> * config/darwin-d.c: New file.
>
> gcc/d/ChangeLog:
>
> * d-target.def (d_minfo_section): New hook.
> (d_minfo_start_name): New hook.
> (d_minfo_end_name): New hook.
> * modules.cc: Include d-target.h.
> (register_moduleinfo): Update to use new targetdm hooks.
> ---
>  gcc/config.gcc|  2 ++
>  gcc/config/darwin-d.c | 49 +++
>  gcc/config/elfos.h|  6 ++
>  gcc/config/t-darwin   |  3 +++
>  gcc/d/d-target.def| 25 ++
>  gcc/d/modules.cc  | 14 ++---
>  gcc/doc/tm.texi   | 20 ++
>  gcc/doc/tm.texi.in|  6 ++
>  8 files changed, 122 insertions(+), 3 deletions(-)
>  create mode 100644 

Re: [PATCH] Add gdc support to macOS

2020-07-11 Thread ciel via Gcc-patches
Iain-san,

I have not heard (or do not find) any news for one year after your reply 500657.
I have just tried my patch for gcc 10.1.0 release and the patch just
worked, which is unexpected for me...

How is your patch going?

T. Yamada

> On Tue, 14 May 2019 at 17:05, ciel  wrote:
> >
> > Dear GCC team,
> >
> > I have succeeded to build gdc on macOS. I firstly submitted the patch
> > to https://github.com/Homebrew/homebrew-core/pull/39712 but I was
> > encouraged to submit to gcc team instead. So could you take a look at
> > it?
> >
>
> Some of it looks on par with work I've already done on this port.
>
> I don't think that the sections/osx.d module is the right way to go,
> particularly if you want proper shared library support.
>
> As for the name of the section, this should be handled by a D language
> target hook.
>
> I'll post patches for this soon once the rough edges have been sorted out.
>
> Thanks,
> --
> Iain


[PATCH] Add gdc support to macOS

2019-05-14 Thread ciel
Dear GCC team,

I have succeeded to build gdc on macOS. I firstly submitted the patch
to https://github.com/Homebrew/homebrew-core/pull/39712 but I was
encouraged to submit to gcc team instead. So could you take a look at
it?

Best Regards,
T. Yamada


gcc9-gdc.patch
Description: Binary data