linuxdcpp-team team mailing list archive
-
linuxdcpp-team team
-
Mailing list archive
-
Message #01470
[Bug 600806] Re: (Open)Solaris doesn't use x64 TTH code
I'm glad it works for you, but I'm not sure if including that in
linuxdcpp would be appropriate. I imagine people using OpenSolaris with
32 bit CPUs wouldn't want 64 bit binaries. You can always pass custom
flags via the CXXFLAGS and LDFLAGS environment variable instead of
hardcoding them in SConstruct.
--
(Open)Solaris doesn't use x64 TTH code
https://bugs.launchpad.net/bugs/600806
You received this bug notification because you are a member of LinuxDC++
Team, which is subscribed to LinuxDC++.
Status in Linux DC++: Confirmed
Bug description:
Using rev 374, OpenSolaris b134.
TigerHash.cpp, chunk of code from line 26:
#ifdef _WIN32
#if defined(_M_X64)
#define TIGER_ARCH64
#endif
#if !(defined(_M_IX86) || defined(_M_X64))
#define TIGER_BIG_ENDIAN
#endif
#else // _WIN32
#if defined(__x86_64__) || defined(__alpha)
#define TIGER_ARCH64
#endif
#if !(defined(__i386__) || defined(__x86_64__) || defined(__alpha))
#define TIGER_BIG_ENDIAN
#endif
#endif // _WIN32
Apparently none of that matches my OSol box, and we end up using the slower 32-bit code.
If I add another line just forcing "#define TIGER_ARCH64", I get much better hashing speed. Using the normal code, I get around 58MB/s, with that define added, more like 81MB/s. Still not great, but its something.
Follow ups
References