[PATCH] [X86] Add HasAVX512 patterns for SS SSE intrinsics.

Quentin Colombet qcolombet at apple.com
Thu May 7 15:41:52 PDT 2015


>   I don't think that we should add all SSE intrinsics to AVX512 now.


I kind of disagree.
When AVX512 is available we have additional registers we can use for those. I believe we should do the same thing as we do for thumb, i.e., choose the most permissive instruction encoding then shorten the encoding if possible (like the Thumb2SizeReduction pass).

For the long term we may want to find a better solution for such case (same instruction available with different encoding based on extension) to avoid the explosion in size of the description.

What do you think?

Cheers,
-Quentin


http://reviews.llvm.org/D9362

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






More information about the llvm-commits mailing list