[PATCH][X86] Factor out new helper getPSHUFB
Adam Nemet
anemet at apple.com
Mon Mar 24 22:52:57 PDT 2014
I found three implementations of this. This splits it out into a new function
and uses it from the three places.
My plan is to add a fourth use when lowering a vector_shuffle:v16i16.
Compared the assembly output of test/CodeGen/X86 before and after.
The only change is due to how the first PSHUFB was generated in
LowerVECTOR_SHUFFLEv8i16. If the shuffle mask specified undef (i.e. -1), the
old implementation would write -1 * 2 and -1 * 2 + 1 (254 and 255) in the
control mask. Now we write 0x80. These are of course interchangeable since
bit 7 decides if a constant zero is written in the result byte. The other
instances of this code used 0x80 consistently.
Related to <rdar://problem/16167303>
Adam
-------------- next part --------------
A non-text attachment was scrubbed...
Name: X86-Factor-out-new-helper-getPSHUFB.patch
Type: application/octet-stream
Size: 6564 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140324/76d96811/attachment.obj>
More information about the llvm-commits
mailing list