D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 3354 - invalid number of args accepted for 1/2 arg floating point instructions
Summary: invalid number of args accepted for 1/2 arg floating point instructions
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: dmd (show other issues)
Version: D2
Hardware: x86 All
: P2 minor
Assignee: yebblies
URL:
Keywords: accepts-invalid, iasm, pull
Depends on:
Blocks:
 
Reported: 2009-10-01 00:29 UTC by Don
Modified: 2015-06-09 05:11 UTC (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Don 2009-10-01 00:29:53 UTC 
This garbage compiles, but shouldn't.

----
void main() {
 double x;
  asm {
  fld x, ST(6);
 }
}
Comment 2 github-bugzilla 2012-02-20 00:58:23 UTC 
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/a6d8be2ab3092d7d7e0d159a8cd0bdac7c234176
Merge pull request #443 from yebblies/issue3354

There is no two argument version of fstp
(That form is in some of the old Intel manuals, but it's redundant).
Comment 3 github-bugzilla 2012-02-20 04:18:59 UTC 
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/37631fc1eae9884d05f6e8d3c6f381dfb69bfdc9
Merge pull request #444 from yebblies/issue3354

Missed one.
Comment 4 github-bugzilla 2012-02-22 19:10:50 UTC 
Commit pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/15303d5c0a5220aad498924f722299885fe01b66
Merge pull request #743 from yebblies/issue3354

Issue 3354 - asm fld x, ST(6); accepted
Comment 5 github-bugzilla 2012-02-22 22:32:05 UTC 
Commit pushed to dmd-1.x at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/47487e17d1c0a386f056081ceb26e69033e7906f
fix Issue 3354 - invalid number of args accepted for 1/2 arg floating point instructions