[Bug target/36720] New: ia64_split_tmode_move doesn't work on little endian

[Bug target/36720] New: ia64_split_tmode_move doesn't work on little endian

[hjl dot tools at gmail dot com]

Jim, Steve, does ia64_split_tmode_move work on little endian?


-- 
           Summary: ia64_split_tmode_move doesn't work on little endian
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl dot tools at gmail dot com
GCC target triplet: ia64-unknown-linux-gnu


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

[Bug target/36720] ia64_split_tmode_move doesn't work on little endian

[hjl dot tools at gmail dot com]

------- Comment #1 from hjl dot tools at gmail dot com  2008-07-03 17:17 -------
There are

       {
          REAL_VALUE_TYPE r;
          unsigned HOST_WIDE_INT p[2];
          long l[4];  /* TFmode is 128 bits */

          REAL_VALUE_FROM_CONST_DOUBLE (r, in); 
          real_to_target (l, &r, TFmode);

          if (FLOAT_WORDS_BIG_ENDIAN)
            {
              p[0] = (((unsigned HOST_WIDE_INT) l[0]) << 32) + l[1];
              p[1] = (((unsigned HOST_WIDE_INT) l[2]) << 32) + l[3];
            }
          else
            {
              p[0] = (((unsigned HOST_WIDE_INT) l[3]) << 32) + l[2];
              p[1] = (((unsigned HOST_WIDE_INT) l[1]) << 32) + l[0];
            }
          out[0] = GEN_INT (p[0]);
          out[1] = GEN_INT (p[1]);

real_to_target converts array of long in the correct endian. Do we
need to adjust endian again?


-- 


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

[Bug target/36720] ia64_split_tmode_move doesn't work on little endian

[hjl dot tools at gmail dot com]

------- Comment #2 from hjl dot tools at gmail dot com  2008-07-03 18:47 -------
A patch is posted at

http://gcc.gnu.org/ml/gcc-patches/2008-07/msg00230.html


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2008-
                   |                            |07/msg00230.html


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

[Bug target/36720] ia64_split_tmode_move doesn't work on little endian

[hjl at gcc dot gnu dot org]

------- Comment #3 from hjl at gcc dot gnu dot org  2008-07-07 00:34 -------
Subject: Bug 36720

Author: hjl
Date: Mon Jul  7 00:34:16 2008
New Revision: 137547

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=137547
Log:
2008-07-06  H.J. Lu  <hongjiu.lu@intel.com>

        PR target/36720
        * config/ia64/ia64.c (ia64_split_tmode): Fix typo in TImode
        constant for little endian.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/ia64/ia64.c


-- 


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

[Bug target/36720] ia64_split_tmode_move doesn't work on little endian

[hjl dot tools at gmail dot com]

------- Comment #4 from hjl dot tools at gmail dot com  2008-07-07 00:41 -------
Fixed.


-- 

hjl dot tools at gmail dot com changed:

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


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

[Bug target/36720] ia64_split_tmode_move doesn't work on little endian

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.4.0


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