New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[dmd-cxx]: Fix the auto-tester and copy dmd to $(GENERATED)/$(OS)/$(BUILD)/$(MODEL) #7953
Conversation
Thanks for your pull request, @wilzbach! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. |
6c1ec1a
to
dc71e70
Compare
Hmm, Phobos still fails to build.
@dkgroot you mentioned on Slack that We could also just stub out edit: -> see dlang/phobos#6220 |
@wilzbach With the current version (ie: dmd + druntime + phobos using branch 'dmd-cxx'), compilation runs fine (still), using: running the unittests for druntime, i get:
Meaning something goes wrong in catching the thrown exception. running the unittests for phobos:, i get:
So there still is a little work to do. |
@ibuclaw Are you able to pass the druntime/phobos 'dmd-cxx' branch unittests using gdc ? |
I can report back in a couple hours on that. Upstream gdc itself mirrors the 2.076.1 branch - plus some platform updates. |
@dkgroot - Actually, I don't think I can say because there's so many things in dmd's Judging from this stack trace you gave.
Problems with |
The version of druntime/phobos that is in gdc's tree does pass all unittests in CI. https://semaphoreci.com/d-programming-gdc/gdc/branches/master/builds/512 |
@wilzbach Have you been able to run the dmd-cxx unittests and/or found the cause for the va_args issue ? I have been looking, but cannot find the cause. |
Ah, bugger. Looks like the assert failure in the auto-tester is issue 15149. But there's a typo in one of the conditions. PR here for testing: D-Programming-GDC/gdc#629 |
This segfault happens because |
With all referenced PRs merged in, I think the auto-tester should be able to get as far as passing the druntime and phobos unittests. |
Windows (dmc) has a very interesting error message:
Looks like there's a bug in dmc to me, as if it overflows on |
dc71e70
to
0532a2d
Compare
Rebased as other PRs have been merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Druntime and Phobos builds in the auto-tester.
Testsuite fails, but then again that's fine, as I don't think any attempt has been made to sync the testsuite yet.
@wilzbach - Let's merge so that it isn't sitting around. |
This should ideally fix the auto-tester build for
dmd-cxx
as Travis is already passing too.See also: #7595