-Inspected a AVX code block added by someone in early Feb. This was never used
authorBruno Cardoso Lopes <bruno.cardoso@gmail.com>
Fri, 22 Jul 2011 00:15:07 +0000 (00:15 +0000)
committerBruno Cardoso Lopes <bruno.cardoso@gmail.com>
Fri, 22 Jul 2011 00:15:07 +0000 (00:15 +0000)
commit6683efb4cdb12785d19100e69f597156e434b9f4
tree8c03db55bfd47406fd825d674f26fe1000f7615a
parent08b076cc968be1c3d1f62f484e3620fca06877c1
-Inspected a AVX code block added by someone in early Feb. This was never used
and was actually very wrong, fix it and make it simpler. Also remove the
ConcatVectors function, which is unused now.

- Fix a introduction of useless nodes in r126664 and r126264. The
VUNPCKL* should never be introduced cause we don't want duplicate
nodes for 128 AVX and non-AVX modes, the actual instruction
difference only exists during isel, but not for target specific DAG
nodes. We only introduce V* target nodes when there is no 128-bit
version already there.

- Fix a fragile test and make it more useful.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@135729 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/X86/X86ISelLowering.cpp
lib/Target/X86/X86ISelLowering.h
test/CodeGen/X86/SIMD/notvunpcklpd.ll
test/CodeGen/X86/SIMD/notvunpcklps.ll