[Bug other/34835] New: splay-tree doesn't support 64bit value on 32bit host

[Bug other/34835] New: splay-tree doesn't support 64bit value on 32bit host

[hjl at lucon dot org]

splay-tree.h has

#ifndef _WIN64
  typedef unsigned long int libi_uhostptr_t;
  typedef long int libi_shostptr_t;
#else
  typedef unsigned long long libi_uhostptr_t;
  typedef long long libi_shostptr_t;
#endif

...

/* Use typedefs for the key and data types to facilitate changing
   these types, if necessary.  These types should be sufficiently wide
   that any pointer or scalar can be cast to these types, and then
   cast back, without loss of precision.  */
typedef libi_uhostptr_t splay_tree_key;
typedef libi_uhostptr_t splay_tree_value;

The problem is on 32bit host libi_uhostptr_t is defined as long and it
don't work long long, which is used by BFD and leads to the linker
bug:

http://www.sourceware.org/bugzilla/show_bug.cgi?id=5303


-- 
           Summary: splay-tree doesn't support 64bit value on 32bit host
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl at lucon dot org


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

[Bug other/34835] splay-tree doesn't support 64bit value on 32bit host

[hjl dot tools at gmail dot com]

------- Comment #1 from hjl dot tools at gmail dot com  2008-02-09 16:23 -------
A patch is posted at

http://sourceware.org/ml/binutils/2008-02/msg00068.html


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://sourceware.org/ml/bin
                   |                            |utils/2008-02/msg00068.html


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

[Bug other/34835] splay-tree doesn't support 64bit value on 32bit host

[hjl.tools at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=34835

H.J. Lu <hjl.tools at gmail dot com> changed:

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

--- Comment #3 from H.J. Lu <hjl.tools at gmail dot com> ---
Fixed on binutils.

[Bug other/34835] splay-tree doesn't support 64bit value on 32bit host

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=34835

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |internal-improvement

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
r0-116114 changes splay_tree_key/splay_tree_value to use uintptr_t directly.
Since binutils no longer uses splay-tree, is this a bug that we need to worry
about?