www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 6097] New: SSSE3 instructions in inline assembler cause error using DMD from git master

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6097

           Summary: SSSE3 instructions in inline assembler cause error
                    using DMD from git master
           Product: D
           Version: D2
          Platform: x86
        OS/Version: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: kai redstar.de



Created an attachment (id=991)
Patch with instruction size information for 3 byte opcodes

In the inline assembler, SSSE3 instructions produce an error if used with MMX
registers. E.g. the program

void main()
{
    asm {
      pshufb MM0, MM1;
    }
}

causes the following error message:

actual: 3, calc: 5
code 003E0A08: nxt=00000000 op=f38 flg=40 rm=c1=3,0,1
Internal error: backend\cod3.c 4448

Everything seems to work if you use only XMM registers.

The root cause seems to be that there is no instruction size stored for 3 byte
opcodes (opcodes starting with 0F 38 and 0F 3A).

The attached patch tries to correct the problem. With the patch applied, some
more instructions could be added to the inline assembler (popcnt, crc32,
movbe).

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 03 2011
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6097


Shahid <hughes4 gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hughes4 gmail.com



Hello, your patch breaks the mandatory prefix check for 2 byte ops such as
PXOR 66 0F EF /r ( SSE2 )

I've edited your patch and included SSE4.1 and 4.2 sizes as well as proper REX
prefix for 4 byte ops (PEXTRQ/PINSRQ)

I also cleaned up the table so that exceptions to the default size would stand
out more by glancing at the code. ( hope you like it :P )

I have a branch with sse4_1 added in my fork on github, but I've not tested it
enough to be confident to issue a pull request.

https://github.com/Govelius/dmd/commits/sse4_1

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 05 2011
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6097


Shahid <hughes4 gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------

           obsolete|                            |



Created an attachment (id=993)
kai redstar.de patch + fixes

Includes kai redstar.de patch + additional fixes

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 05 2011
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6097


Walter Bright <bugzilla digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bugzilla digitalmars.com
         Resolution|                            |FIXED



19:14:01 PDT ---
https://github.com/D-Programming-Language/dmd/commit/d3f5c9c2bd143bc0037640465250603b244431c7

https://github.com/D-Programming-Language/dmd/commit/9a8b9b12386cef1a4423bfc14ea3159a50e2623a

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jul 15 2011