[PATCH] [X86] Replace (v)palignr intrinsics with generic shuffles (Clang)

Craig Topper craig.topper at gmail.com
Thu Mar 12 11:57:48 PDT 2015


We've always been sending shuffles to the backend. We just generated the shuffles in CGBuiltin instead of the header.

I'm not sure I like completely losing the type system on the immediate. Theoretically with the code in CGBuiltin we could at least get a truncation warning if the immediate was larger than a byte. Though I'm not sure that warning is on by default. Really I wish we could check the immediates for illegal values on all of these macros and deliver nice messages to the user. I think gcc does check a lot of them.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D8301

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






More information about the cfe-commits mailing list