[PATCH] [X86] Also create v2f32 FMIN/FMAX nodes from fcmp/select.

Ahmed Bougacha ahmed.bougacha at gmail.com
Fri Dec 5 16:20:08 PST 2014


We also learned to legalize v2f32 FMIN/FMAX nodes later on.
I tried to also support v3f32, which might also happen often, but they
were already widened to v4f32 by the generic split-and-rebuild-vector
legalization.

This happens in the HINT benchmark, where the SLP-vectorizer created
v2f32 fcmp/select code.  The "correct" solution would have been to
teach the vectorizer code model that v2f32 isn't legal (because really,
it isn't), but if we can vectorize we might as well do so.

http://reviews.llvm.org/D6557

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/sse-minmax.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6557.17007.patch
Type: text/x-patch
Size: 3150 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141206/c89a45e8/attachment.bin>


More information about the llvm-commits mailing list