adelikat / snes9x-rr

Automatically exported from code.google.com/p/snes9x-rr
0 stars 0 forks source link

Super Donkey Kong (J) (V1.1) in Snes9x 1.51 r66 #21

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.
2.
3.

What is the expected output? What do you see instead?
Super Donkey Kong (J) (V1.1) in Snes9x 1.51 r66.
The Background is broken.
But in 1.43 r66 is not.

What version of the product are you using? On what operating system?
Snes9x 1.51 r66

Please provide any additional information below.
A screenshot for you.
Sorry about my English.My English is not good.

Original issue reported on code.google.com by kevinshe...@gmail.com on 2 Oct 2009 at 10:01

Attachments:

GoogleCodeExporter commented 9 years ago
First of all, thank you for your report, sir.

It occurs because of optimization fault of Visual Studio (2008). I can prevent 
this
regression by putting '#pragma optimize("", off)' at the top of cpu.cpp. 
However, I
think it's not a real fix, so I'm investigating what line causes such a strange 
issue.

For now, use revision 61 instead of the latest one. The difference isn't so 
large.

Original comment by gochaism on 2 Oct 2009 at 2:51

GoogleCodeExporter commented 9 years ago

Original comment by gochaism on 2 Oct 2009 at 2:52

GoogleCodeExporter commented 9 years ago
The cause is in memmap.h, it seems.

Original comment by gochaism on 2 Oct 2009 at 3:01

GoogleCodeExporter commented 9 years ago
More accurately, getset.h

Original comment by gochaism on 2 Oct 2009 at 3:10

GoogleCodeExporter commented 9 years ago
I don't know why, but the glitch occurs because of S9xGetWord. To help 
investigating
for the details, I dumped the optimized assembly by using code listing option of
VC2008. http://pastebin.ca/1589620

Original comment by gochaism on 2 Oct 2009 at 4:07

GoogleCodeExporter commented 9 years ago
I fixed the issue in revision 69 without hacky optimization switch.

Original comment by gochaism on 2 Oct 2009 at 10:59