Index
> Compiler Internals > HEAP keyword broken? (64-bit) |
| Author |
|
|
nts94 02 Apr 2013, 21:48
Well, the problem is in FORMATS.INC, near peplus_heap label. The only difference between the 32 bit and the 64 bit is that edx gets trashed in the later, and then it crashed when "cmp eax,[edx+70h]" is executed at line 1545.
Comparing that routine with peplus_stack, it seems that edx is mistaken, it should be ecx (but I could be wrong, I'm not very comfortable with fasm sources yet) |
|||
02 Apr 2013, 21:48 |
|
|
bitRAKE 02 Apr 2013, 22:24
Code: cmp edx,[ecx+74h] ja value_out_of_range jb instruction_assembled cmp eax,[edx+70h] ja value_out_of_range jmp instruction_assembled |
|||
|
02 Apr 2013, 22:24 |
|
|
Tomasz Grysztar 03 Apr 2013, 12:31
Thank you for finding this out. I'm applying the correction to 1.71.10 release (coming soon).
|
|||
|
03 Apr 2013, 12:31 |
|
|
l_inc 03 Apr 2013, 14:02
Tomasz Grysztar
You might want to include a fix for the following one either. Code: ;should display 'a', but fails with undefined instruction struc x { display .,13,10 } 'a' x It is related to the bug I reported before, which IMHO was not completely resolved. _________________ Faith is a superposition of knowledge and fallacy |
|||
|
03 Apr 2013, 14:02 |
|
|
Tomasz Grysztar 03 Apr 2013, 14:12
The invocation of "struc" macro needs to have a valid name token as a label - no other token (or combination of tokens) is allowed there.
Code: macro x { display 13,10 } struc x { display `.,13,10 } x ; valid invocation of macro a: x ; another valid invocation of macro a x ; valid invocation of struc 'a' x ; not an invocation of either If you want to report new bugs, please create new topic for each report - the moderator will make it sticky when it's valid. |
|||
|
03 Apr 2013, 14:12 |
|
|
l_inc 03 Apr 2013, 14:17
Tomasz Grysztar
Well. Because this one had a potential to initiate a discussion, I didn't want to report it, until I have more free time. 
Quote: If you want to report new bugs, please create new topic for each report - the moderator will make it sticky when it's valid. OK. I hoped one post would be enough, but obviously I will need to create a separate topic for multiple deficiencies. _________________ Faith is a superposition of knowledge and fallacy |
|||
|
03 Apr 2013, 14:17 |
|
|
DOS386 04 Apr 2013, 01:45
> Thank you for finding this out. I'm applying the correction to 1.71.10 release (coming soon).
It's out: Code: version 1.71.10 (Apr 03, 2013) [-] Fixed a crashing "heap" directive in 64-bit PE format. Tomasz Grysztar wrote: If you want to report new bugs, please create new topic for each report - the moderator will make it sticky when it's valid. Could you please also look at: http://board.flatassembler.net/topic.php?t=15279 (sugg, not BUG) _________________ Bug Nr.: 12345 Title: Hello World program compiles to 100 KB !!! Status: Closed: NOT a Bug |
|||
|
04 Apr 2013, 01:45 |
|
< Last Thread | Next Thread > |
Forum Rules:
|