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

x64 DMD builds has rendering problems. #35

Closed
p0nce opened this issue May 26, 2015 · 19 comments
Closed

x64 DMD builds has rendering problems. #35

p0nce opened this issue May 26, 2015 · 19 comments
Labels
Bug Reproduced bug.

Comments

@p0nce
Copy link
Collaborator

p0nce commented May 26, 2015

Text is odd looking and no lights.
@p0nce p0nce added the Bug Reproduced bug. label May 26, 2015
@p0nce
Copy link
Collaborator Author

p0nce commented May 26, 2015

Disappears in debug build.

@p0nce
Copy link
Collaborator Author

p0nce commented May 26, 2015

Looks like stack corruption, you can crash VSTHost x64 by clearing the Knob.onDraw method.

@p0nce p0nce mentioned this issue May 26, 2015
Closed
@p0nce p0nce added this to the 1.0 milestone Jun 17, 2015
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 17, 2015

bug

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 17, 2015

Not affected by --combined

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 17, 2015

Removing -O on gfm:image makes the bug disappear. Probably incorrect code in stb_truetype translation. Aggravated by the fact we tweak SSE and FPU settings in all callbacks.

p0nce pushed a commit to d-gamedev-team/gfm that referenced this issue Jun 17, 2015
Disable optimization for gfm:image until a proper fix is found for Au…
f9b7c50 
…burnSounds/Dplug#35
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 18, 2015

There is another similar visual bug that disappear when removing -O on dplug:gui.

p0nce pushed a commit that referenced this issue Jun 18, 2015
Work-around issue #35 (rendering problem)
10f602f
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 18, 2015

Rewrote softCircle without fixed point to avoid something looking like a codegen bug.
50bb78a
Avoids the 2nd problem.

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 18, 2015

1st problem work-arounded just by screwing around.
d-gamedev-team/gfm@d4526df

TODO: identify if codegen bug or not.

@p0nce p0nce closed this as completed Jun 18, 2015
@p0nce p0nce reopened this Jun 19, 2015
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 19, 2015

notfixed

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 19, 2015

Two problems here:

  • text is still not right
  • the bargraph leds are ill-positionned compared to 32-bits

p0nce added a commit that referenced this issue Jun 19, 2015
@p0nce
Work-around strange codegen problem with x0 in Bargraph.onDraw
07f8c7a 
One less problem in #35
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 23, 2015

Fixed with d-gamedev-team/gfm@fa50d24

@p0nce p0nce closed this as completed Jun 23, 2015
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 24, 2015

The problem is still around and need a reduced test-case.

@p0nce p0nce reopened this Jun 24, 2015
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 24, 2015

Known bug since April:
https://issues.dlang.org/show_bug.cgi?id=14510
https://issues.dlang.org/show_bug.cgi?id=14480

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 24, 2015

Work-arounded again 7e18fe4

@p0nce
Copy link
Collaborator Author

p0nce commented Jul 1, 2015

To test with DMD 2.068-b2 when it's out, and then remove dplug:quanrantine

@p0nce
Copy link
Collaborator Author

p0nce commented Jul 26, 2015

DMD 2.068-b2 is out

@p0nce
Copy link
Collaborator Author

p0nce commented Aug 17, 2015

Filed here:
https://issues.dlang.org/show_bug.cgi?id=14896

@p0nce
Copy link
Collaborator Author

p0nce commented Aug 23, 2015

This last issue can be removed by replacing synchronized blocks in font.d.

@p0nce
Copy link
Collaborator Author

p0nce commented Aug 24, 2015

Used a mutex, font.dis back in dplug:gui.--combined is now supported to the detriment of linux support.

@p0nce p0nce closed this as completed Aug 24, 2015
@p0nce p0nce mentioned this issue Sep 2, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Reproduced bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@p0nce