[PATCH] D13505: [X86] Fix bad treatment of multi-lane blends in BUILD_VECTORtoBlendMask()

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 02:47:40 PDT 2015


mkuper created this revision.
mkuper added reviewers: filcab, chandlerc, aivchenk.
mkuper added subscribers: qcolombet, llvm-commits.

This is a replacement for the patch in D13207.
It turns out that there are actually separate two bugs. One is the bug that D13207 originally handled.
The other is that the number of lanes it computes may not actually be 1 or 2. See the AVX case of the constant_pblendvb_avx2 test for an example. I'm not sure how exactly it should behave, so I've decided to pessimize it back to safety.

http://reviews.llvm.org/D13505

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/vector-blend.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13505.36721.patch
Type: text/x-patch
Size: 7624 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151007/e12b44d9/attachment.bin>


More information about the llvm-commits mailing list