[PATCH] Remove spurious mask operations from AArch64 add->compares on 16 and 8 bit values

Louis Gerbarg lgg at apple.com
Thu Aug 21 14:21:43 PDT 2014


This is the patch I discussed a while back in http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-July/074444.html. It eliminates spurious ands on 8 and 16 bit operations that have been promoted to 32 bit that have no material impact on these test cases they are fed into. It remove them in several cases that have come up in testing. It is entirely possible there are missed cases due to a slightly differently configured DAG, but adding those should be simple if we have examples of them. This patch has been tested against LNT on AArch64 hardware and includes tests that were randomly generated using on device data as well as ones derives from actual bugs.

Louis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Remove-spurious-mask-operations-from-AArch64-add-com.patch
Type: application/octet-stream
Size: 14784 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140821/d6449f06/attachment.obj>


More information about the llvm-commits mailing list