[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4D8C5C24.2010402@bredband.net>
Date: Fri, 25 Mar 2011 10:11:00 +0100
From: magnum <rawsmooth@...dband.net>
To: john-dev@...ts.openwall.com
Subject: Re: john scalability
On 2011-03-24 02:59, Solar Designer wrote:
> I'd appreciate it if you submit this fix as a separate patch (on top of
> jumbo), _not_ including the UTF-8 stuff into the same patch. I might
> want to put out a "stable" version first.
Even when including the UTF-8 stuff, you could/should use "#ifdef
FMT_UTF8" (this is defined by the UTF-8 patch) in the assembler files,
for a version that optionally keeps that bitshift optimisation.
Later, I/you/we might end up with UTF-8 aware code that still use the
optimisation when the --utf8 option is not used. I really try to avoid
performance impacts when not using it. Though I guess this means having
(yet another) two of each arch-dependant function. Maybe the speedup is
not worth it?
thanks
magnum
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.
