[PATCH] D26890: SHA1: unroll loop in hashBlock.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 19 16:50:53 PST 2016


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

LGTM, Thanks!

In https://reviews.llvm.org/D26890#600802, @ruiu wrote:

> If I do not unroll the loop, computing a hash on a file worsened from 4.16 seconds to 5.77, so there's definitely an impact.


Please file a PR on llvm.org with the non-unrolled version of this code, and CC me. The optimization folks should look at this!



================
Comment at: lib/Support/SHA1.cpp:82
+  B = rol(B, 30);
+}
+
----------------
Please use static free function if there is no need to access the state of the object.


https://reviews.llvm.org/D26890





More information about the llvm-commits mailing list