[PATCH] Make the byte-shift SSE intrinsics emit vector shuffles which we know the backend can handle.

[Craig Topper]

Sorry I missed this earlier.

I'm all for removing unnecessary builtins. Does the backend lower these shuffles to the same instruction the builtin does? Do we have tests for that in the llvm side?

Do you plan to remove the intrinsics from llvm in favor of the shuffles?

The casts inside the diagnostic pragmas seem inconsistent between the two macros. Was that intentional.

Is there already an emmintrin test file this could be added to? Maybe just put them in sse-builtins.c


http://reviews.llvm.org/D7199

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