[PATCH] [X86] replace vinsertf128 intrinsics with generic shuffles

[Craig Topper]

Macros are needed when clang's integer constant evaluation is being used to create the inputs to the vector shuffle.

I believe with the inline version you've used the IR will contain two shuffles and the optimizers in the backend will have to reduce it.

I think I'd prefer the macro approach and only one shuffle with math on the immediates.


http://reviews.llvm.org/D8088

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