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.