Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 8 Jul 2012 08:07:32 -0500
From: "jfoug" <jfoug@....net>
To: <john-dev@...ts.openwall.com>
Subject: RE: ARCH_ALLOWS_UNALIGNED in generic build (was: Build error for generic)

>From: Solar Designer [mailto:solar@...nwall.com]
>In md4.c and md5.c, we have:
>
>#if defined(__i386__) || defined(__x86_64__) || defined(__vax__)

Changing the above line, to 

#if defined(__i386__) || defined(__x86_64__) || defined(__vax__) || defined
(_M_IX86) || defined(_M_X64)

Caused my vc build of the new dyna_1010, to go from 2010k, to 2391k on my
system.  The first was set in md4.c, md5.c, md5_go.c (which my dynamic
uses), and md5_eq.c

The 'wrong' code worked, but it was doing byte/byte swapping, from Intel
format, back to Intel format.  Not too efficient when compared to a int
assignment. There is a comment, listing this is just an optimization, and
nothing 'fails' if it gets it wrong. But the #define is woefully lacking.
Even the way I have it, it is greatly lacking.

Jim.

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.