[PATCH] D28747: [X86] Don't create VBROADCAST nodes with 256-bit or 512-bit input types

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

In https://reviews.llvm.org/D28747#677154, @craig.topper wrote:

> Is using an fp broadcast for an integer operation for AVX1 a good idea. Is there a stack cross penalty for that on Sandy Bridge?


For 256-bit cases using broadcastss/broadcastsd is a definite win - plus nearly all the 256-bit AVX1 operations are in the fp-domain (including how we lower v8i32/v4i64 shuffles). For 128-bit cases its less clear.


Repository:
  rL LLVM

https://reviews.llvm.org/D28747