[PATCH] [SLPVectorizer] Reorder operands of shufflevector if it can result in a vectorized code.

[Karthik Bhat]

Hi Michael,
I went through the link http://reviews.llvm.org/D6675. The swap operation is different for these 2 patches. The patch in the http://reviews.llvm.org/D6675 tries to swap across lanes and is valid only for interger operands.

In this patch  though we swap within the same lane when the operand is commutative so it can be applied to floating point operands as well because float addition etc are commutative but NOT necessarily transitive.
I feel having seperate function to handle this case would be better. But please let me know if you feel otherwise; will see if i can get some common parts out for the 2 patches.

Thanks and Regards
Karthik Bhat


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6677

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/