Skip to content
This repository was archived by the owner on Jun 20, 2019. It is now read-only.

Update frontend to DDMD v2.076#550

Merged
ibuclaw merged 11 commits intoD-Programming-GDC:masterfrom
ibuclaw:ddmd
Jun 18, 2018
Merged

Update frontend to DDMD v2.076#550
ibuclaw merged 11 commits intoD-Programming-GDC:masterfrom
ibuclaw:ddmd

Conversation

@ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Sep 24, 2017

Nothing to say here, only: 😀 😢 🤔 🤓 😬 😨 😱

@ibuclaw
Copy link
Member Author

ibuclaw commented Sep 28, 2017

It looks like we are passing the testsuite now. :-)

@MartinNowak
Copy link

Congrats, and thanks for your amazing work.

@ibuclaw
Copy link
Member Author

ibuclaw commented Sep 30, 2017

This should go in a new branch perhaps. Bootstrapped builds are also not yet working.

@jpf91
Copy link
Contributor

jpf91 commented Oct 1, 2017

This should go in a new branch perhaps. Bootstrapped builds are also not yet working.

OK. I just reviewed the changes to the GCC patches, configure/makefiles and glue code and these look fine so far.

@PetarKirov
Copy link

PetarKirov commented Dec 31, 2017

What's current status @ibuclaw? What remains to be done until GDC can finally switch to ddmd and you can start to follow dmd's release schedule? I'm hopeful that due to the much smaller front-end diff one day we may be able to setup some automatic merge, build and test of dmd/druntime/phobos pull-requests with GDC on of the CI services that we use. #wishlist2018

@ibuclaw
Copy link
Member Author

ibuclaw commented Jan 1, 2018

Stage 3 builds weren't quite working yet (bootstrapped builds) and it needs a rebase. Probably the only two things that need to be addressed.

@ibuclaw
Copy link
Member Author

ibuclaw commented May 30, 2018

Improvements that should be looked into:

  • Link gdc against libgdruntime, not libgphobos.

And things that would save time building:

  • Investigate if libgphobos part can be compiled only during stage3, in the last step after the compiler is build.
  • Investigate if libatomic and/or backtrace bootstrapping can be moved to toplevel configure, as if it's not used by druntime, then it's not required for bootstrapping.

@ibuclaw
Copy link
Member Author

ibuclaw commented May 31, 2018

Link gdc against libgdruntime, not libgphobos.

Which looks like it will require support in the driver, both for linking druntime statically and using the generated libgphobos.spec file for adding all dependencies.

@ibuclaw ibuclaw changed the title [WIP] Test building against ddmd Update frontend to DDMD v2.076 Jun 9, 2018
@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 9, 2018

I think I just have a couple more patches to push to master, then let's just rebase and go with this.

@ibuclaw ibuclaw force-pushed the ddmd branch 3 times, most recently from b05633e to b97bf4f Compare June 16, 2018 15:02
@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 16, 2018

configure: error: GDC is required to build d
make: *** No targets specified and no makefile found. Stop.

@braddr - gdc is required to be installed on the build servers after this is merged. I've been testing with gdc-7 and gdc-8 and all works fine.

Alternatively, you could build with GDC=dmd, should work also, but untested as of yet.

@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 16, 2018

Alternatively, you could build with GDC=dmd, should work also, but untested as of yet.

Though maybe not, as dmd wouldn't understand options like -no-pie.

@ibuclaw ibuclaw force-pushed the ddmd branch 3 times, most recently from 197c7f4 to 07dd3d9 Compare June 17, 2018 19:38
@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 17, 2018

Time to venture the point of no return?

@belka-ew
Copy link
Contributor

belka-ew commented Jun 17, 2018

Time to venture the point of no return?

of @noreturn. Yes 🥇

@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 18, 2018

🙈 🙉 🙊

@ibuclaw ibuclaw merged commit 3600ef7 into D-Programming-GDC:master Jun 18, 2018
@ibuclaw ibuclaw deleted the ddmd branch June 18, 2018 21:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants