<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 19, 2020 at 7:29 AM Jorrit Jongma via rsync <<a href="mailto:rsync@lists.samba.org">rsync@lists.samba.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I've read up some more on the subject, and it seems the proper way to do this with GCC is g++ and target attributes. I've refactored the patch that way, and it indeed uses SSSE3 automatically on supporting CPUs, regardless of the build host, so this should be ideal both for home builders and distros.<br></blockquote><div><br></div><div>Very cool stuff!  Sounds like this should be a nice improvement for x86 systems. I've tweaked your patch just a bit and changed it to where it is disabled by default unless configured with --enable-sse2, just to be extra cautious. Once it gets some validation testing we can change that to be enabled by default.  Thanks!</div><div><br></div><div>As for xxhash support, the current version of that patch is in the patches git repo (it hasn't been merged yet):</div><div><br></div><div><a href="https://git.samba.org/?p=rsync-patches.git;a=blob_plain;f=xxhash.diff;hb=6f3c2cd839ccbae8dc74222f9f8a60aef0156d43">https://git.samba.org/?p=rsync-patches.git;a=blob_plain;f=xxhash.diff;hb=6f3c2cd839ccbae8dc74222f9f8a60aef0156d43</a> <br></div><div><br></div><div dir="ltr"><div><div dir="ltr" class="gmail_signature">..wayne..</div></div></div></div></div>