On Thu, Aug 23, 2018 at 02:40:35PM -0700, Conrad Meyer wrote:
> On Thu, Aug 23, 2018 at 11:19 AM, Alex Richardson
> <arichard...@freebsd.org> wrote:
> > Author: arichardson
> > Date: Thu Aug 23 18:19:33 2018
> > New Revision: 338270
> > URL: https://svnweb.freebsd.org/changeset/base/338270
> >
> > Log:
> >   Don't build skein_block_asm.s if we don't have an as binary
> >...
> > It might also be better to just
> >   compile the C file in userspace since the compiler can the use SSE/AVX.
> 
> FYI, in my (limited) testing, the hand-written assembly has something
> like 50% performance advantage over the reference C implementation,
> even with -march=native, etc.  Using the reference C version is
> totally fine for bootstrapping, but the compiler is not yet smart
> enough to write the assembler versions.

Thanks for doing this.  Sounds like we should keep the option to build
if you have a GNU as around.

-- Brooks

Attachment: signature.asc
Description: PGP signature

Reply via email to