[PATCH] D74621: ValueTracking: Fix crash on weird size bswap

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 08:59:49 PST 2020


nikic added a comment.

Sure, non-byte multiplies doesn't make sense, but byte multiples generally do. However, after checking LangRef we apparently do require the bitwidth to be a multiple of 16 for this intrinsic: https://llvm.org/docs/LangRef.html#llvm-bswap-intrinsics


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74621/new/

https://reviews.llvm.org/D74621





More information about the llvm-commits mailing list