← Back to index | Original Bugzilla link

Bug 3354 – invalid number of args accepted for 1/2 arg floating point instructions

Status: RESOLVED
Resolution: FIXED
Severity: minor
Priority: P2
Component: dmd
Product: D
Version: D2
Platform: x86
OS: All
Creation time: 2009-10-01T00:29:00Z
Last change time: 2015-06-09T05:11:49Z
Keywords: accepts-invalid, iasm, pull
Assigned to: yebblies
Creator: clugdbug

Comments

Comment #0 by clugdbug — 2009-10-01T00:29:53Z

This garbage compiles, but shouldn't. ---- void main() { double x; asm { fld x, ST(6); } }

Comment #1 by yebblies — 2012-02-19T21:33:45Z

https://github.com/D-Programming-Language/dmd/pull/743

Comment #2 by github-bugzilla — 2012-02-20T00:58:23Z

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 by github-bugzilla — 2012-02-20T04:18:59Z

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 by github-bugzilla — 2012-02-22T19:10:50Z

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 by github-bugzilla — 2012-02-22T22:32:05Z

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