flat assembler
Message board for the users of flat assembler.

Index > Compiler Internals > Minor flaw in fasm v1.50

Author
Thread Post new topic Reply to topic
yazper



Joined: 20 Nov 2003
Posts: 4
Location: at home
yazper 16 Dec 2003, 18:27
Hi everyone

I was just playing around with some code after dwl version 1.5 and found that

Code:
        sbb     edx,0
    


gets assembled to

Code:
        sbb     edx,-1
    


I tried to assemble it in version 1.49 and it gave the correct result
So I think Privalov introduced a new bug

thanks

yazper

_________________
--------------------
Light speed is faster then the speed of sound - that's why ppl look smart until they talk!
Post 16 Dec 2003, 18:27
View user's profile Send private message Reply with quote
Tomasz Grysztar



Joined: 16 Jun 2003
Posts: 8357
Location: Kraków, Poland
Tomasz Grysztar 16 Dec 2003, 19:41
Minor flaw? I would rather call it a "serious bug" if it was true (fasm itself wouldn't work correctly after such reassembly). But all it's OK in my test, if it happened to you in some special case, please post your source.
Post 16 Dec 2003, 19:41
View user's profile Send private message Visit poster's website Reply with quote
yazper



Joined: 20 Nov 2003
Posts: 4
Location: at home
yazper 16 Dec 2003, 20:34
Ok. sorry I didnt check it thouroghly.
I use Ollydebug for ...yes well...debugging a sinusgenerator made with a 2-pole filter.
Olly gives me "sbb edx,-1" when using 1.50 and "sbb edx,0" when using 1.49.

Here's the source

The culprit is line40 in "sinusgenerator.fasm"
Dont post about the generator not working, thats why I'm debugging Wink

Plz go easy on my coding...I'm somewhat new to assembler on PC, but I've programmed lots on the Amiga using the (in my opinion) best assembler environment ever made (AsmPro). Maybe you could use some of the ideas from AsmPro?

PS: I can give you screen shots of olly if you want?!

-yazper


Description:
Download
Filename: source.zip
Filesize: 2.23 KB
Downloaded: 849 Time(s)


_________________
--------------------
Light speed is faster then the speed of sound - that's why ppl look smart until they talk!
Post 16 Dec 2003, 20:34
View user's profile Send private message Reply with quote
comrade



Joined: 16 Jun 2003
Posts: 1150
Location: Russian Federation
comrade 16 Dec 2003, 20:45
Assembled just fine here. I check disassembly and its all okay. Maybe you confuse two sbb's together?

_________________
comrade (comrade64@live.com; http://comrade.ownz.com/)
Post 16 Dec 2003, 20:45
View user's profile Send private message Visit poster's website AIM Address Yahoo Messenger MSN Messenger ICQ Number Reply with quote
Tomasz Grysztar



Joined: 16 Jun 2003
Posts: 8357
Location: Kraków, Poland
Tomasz Grysztar 16 Dec 2003, 20:49
This is how that routine does look like in my OllyDbg - as you can see, the "sbb edx,0" is encoded correctly. Perhaps you have mistaken it with the instruction which is at address 4020A3h on this screen, this is "sbb edx,0ffffffffh" from your source.


Description:
Filesize: 6.52 KB
Viewed: 9564 Time(s)

ollydbg.gif


Post 16 Dec 2003, 20:49
View user's profile Send private message Visit poster's website Reply with quote
yazper



Joined: 20 Nov 2003
Posts: 4
Location: at home
yazper 16 Dec 2003, 20:55
Oops Embarassed
I did confuse it.... i'm just making a complete arse of myself Crying or Very sad
Sorry for wasting you time.

-yazper

_________________
--------------------
Light speed is faster then the speed of sound - that's why ppl look smart until they talk!
Post 16 Dec 2003, 20:55
View user's profile Send private message Reply with quote
Display posts from previous:
Post new topic Reply to topic

Jump to:  


< Last Thread | Next Thread >
Forum Rules:
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You can download files in this forum


Copyright © 1999-2024, Tomasz Grysztar. Also on GitHub, YouTube.

Website powered by rwasa.