[PATCH] Optimize memcmp(x, y, n)==0 for small n and suitably aligned x/y.

Eelis eelis at eelis.net
Sun Jan 25 21:17:47 PST 2015


Ah, it wasn't that DL was null, but rather that there were no native integer types specified in the target datalayout for the test. I've added those, and I've also refined the alignment check to take the integer type's preferred alignment into account.

I think it's okay to commit now. Chad: I don't have commit rights, so yes please! Thanks :)


http://reviews.llvm.org/D6952

Files:
  lib/Transforms/Utils/SimplifyLibCalls.cpp
  test/Transforms/InstCombine/memcmp-1.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6952.18738.patch
Type: text/x-patch
Size: 3915 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150126/4a1f9b0d/attachment.bin>


More information about the llvm-commits mailing list