[Bug target/23660] New: Moves from general to MMX registers use unnecessary secondary memory

[Bug target/23660] New: Moves from general to MMX registers use unnecessary secondary memory

[vahur dot sinijarv at cydonia dot ee]

I was recompiling my MMX code on GCC 4 and was surprised to see that none of 
the moves from general regs to MMX (and other way around) were happening 
directly but through intermediate memory location and I could not find a 
combination of optimization options which would disable this behaviour.

I tried to find out what is causing this and this is what I found:

gcc/config/i386/i386.c:
int ix86_secondary_memory_needed (enum reg_class class1, enum reg_class 
class2, enum machine_mode mode, int strict)
...

/* ??? This is a lie. We do have moves between mmx/general, and for mmx/sse2.
But by saying we need secondary memory we discourage the register allocator
from using the mmx registers unless needed. */

if (MMX_CLASS_P (class1) != MMX_CLASS_P (class2)) return true;
... 

If I comment out that "if" line, mmx register assignments from general 
registers will work.
Should the requirement of using secondary memory to move MMX registers between 
general regs be relaxed ?
I do not see the point why one should discourage the register allocator from 
using mmx registers, move through memory is clearly inefficent and enlarges 
resulting code (if the function containing moves is inlined in several places, 
even more so).
I'll attach my source and assembly output from GCC 4.1.0 (30.08.2005 CVS HEAD) 
vs GCC 3.4.4, these are short.

-- 
           Summary: Moves from general to MMX registers use unnecessary
                    secondary memory
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: vahur dot sinijarv at cydonia dot ee
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: i686-pc-mingw32
  GCC host triplet: i686-pc-mingw32
GCC target triplet: i686-pc-mingw32


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23660

[Bug target/23660] Moves from general to MMX registers use unnecessary secondary memory

[vahur dot sinijarv at cydonia dot ee]

------- Additional Comments From vahur dot sinijarv at cydonia dot ee  2005-08-31 17:24 -------
Created an attachment (id=9637)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9637&action=view)
Preprocessed source


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23660

[Bug target/23660] Moves from general to MMX registers use unnecessary secondary memory

[vahur dot sinijarv at cydonia dot ee]

------- Additional Comments From vahur dot sinijarv at cydonia dot ee  2005-08-31 17:34 -------
Created an attachment (id=9638)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9638&action=view)
4.1.0 vs 3.4.4


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23660

[Bug target/23660] Moves from general to MMX registers use unnecessary secondary memory

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-08-31 19:18 -------
The issue is more complicated than you think. 

This is a dup of bug 22076 which has about the same code and issue.

*** This bug has been marked as a duplicate of 22076 ***

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |DUPLICATE


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23660