Skip to content
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.

Sign up for GitHub

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

Jump to bottom

make:Internal error: backend/go.c 242 #176

Closed
timotheecour opened this issue Jan 2, 2017 · 4 comments · Fixed by #179
Closed

make:Internal error: backend/go.c 242 #176

timotheecour opened this issue Jan 2, 2017 · 4 comments · Fixed by #179

Comments

@timotheecour
Copy link
Contributor 

Internal error: backend/go.c 242
make: *** [demos/gtkD/TestWindow/TestStock.o] Error 1
make: *** Waiting for unfinished jobs....
MikeWey added a commit that referenced this issue Jan 3, 2017
@MikeWey
Disable Optimizations for the demo because of dmd issue 14214. #176
1d0839e
@MikeWey
Copy link
Member

MikeWey commented Jan 3, 2017

It's an bug in dmd that is triggered by the demo.
I've disabled the optimizations for dmd to avoid this issue.

https://issues.dlang.org/show_bug.cgi?id=14214

@MikeWey MikeWey closed this as completed Jan 3, 2017
@timotheecour
Copy link
Contributor Author

timotheecour commented Jan 3, 2017 •
edited
Loading

thanks; that's a rather radical fix ; could this be narrowed down to a single or few files? Also how to make this work within a dub executable target?

The linked bug report shows it happens with a very large cardinality TypeTuple; could we instead fix that? Where in the code would that be?

EDIT: happens here:

dmd -O -Isrc -Idemos/gtkD/TestWindow -c demos/gtkD/TestWindow/TestStock.d -ofdemos/gtkD/TestWindow/TestStock.o

NOTE:
ldc doesn't have this bug, and is faster:

ldmd2 -O3 -Isrc -Idemos/gtkD/TestWindow -c demos/gtkD/TestWindow/TestStock.d -ofdemos/gtkD/TestWindow/TestStock.o

timotheecour added a commit to timotheecour/GtkD that referenced this issue Jan 3, 2017
@timotheecour
fix issue gtkd-developers#176
b50e7dc 
fix gtkd-developers#176
@timotheecour timotheecour mentioned this issue Jan 3, 2017
Merged
@timotheecour
Copy link
Contributor Author

#179 fixes this issues and is more efficient.

@timotheecour
Copy link
Contributor Author

after merging #179 could you revert 1d0839e ?

MikeWey added a commit that referenced this issue Jan 4, 2017
@MikeWey
Merge pull request #179 from timotheecour/patch-1
82cf660 
fix issue #176
MikeWey added a commit that referenced this issue Jan 4, 2017
@MikeWey
Revert "Disable Optimizations for the demo because of dmd issue 14214. …
8fb00c6 
…#176"

A better fix was introduced in pull #179.

This reverts commit 1d0839e.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix issue #176 timotheecour/GtkD
2 participants
@MikeWey @timotheecour