Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150210205047.GK23507@brightrain.aerifal.cx>
Date: Tue, 10 Feb 2015 15:50:47 -0500
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: [PATCH 1/2] x86_64/memset: simple optimizations

On Tue, Feb 10, 2015 at 06:30:56PM +0100, Denys Vlasenko wrote:
> "and $0xff,%esi" is a six-byte insn (81 e6 ff 00 00 00), can use
> 4-byte "movzbl %sil,%esi" (40 0f b6 f6) instead.
> [...]

Do you want to go ahead with these patches as-is, or consider some of
the other ideas we discussed off-list like avoiding the 64-bit imul
entirely in the small-n case? If you think that's easy as another
incremental change I'll go ahead with these, but if it's basically
going to be backing out these first changes and replacing them with
something different, I'm not sure if it makes sense to commit one then
the other.

Rich

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.