[PATCH] [X86] Fix bug in new build_vector v4x32 lowering

Michael Kuperstein michael.m.kuperstein at intel.com
Sun Nov 23 03:43:14 PST 2014


Hi andreadb, qcolombet,

The new build_vector lowering misses a case where:

1) A single extracted element is used twice.
2) The lower of the two non-zero indexes should be preserved, and the higher should be used for the dest mask.

This causes a crash, since the source value for the insertps ends-up uninitialized.

http://reviews.llvm.org/D6377

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/sse41.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6377.16536.patch
Type: text/x-patch
Size: 2145 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141123/3bb4fd5b/attachment.bin>


More information about the llvm-commits mailing list