[PATCH] D28216: AVX-512: Truncate with unsigned saturation.

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 06:29:45 PST 2017


delena marked an inline comment as done.
delena added inline comments.


================
Comment at: ../lib/Target/X86/X86ISelLowering.cpp:31144
+  // the element size of the destination type.
+  return (C == ((uint64_t)1 << VT.getScalarSizeInBits()) - 1) ?
+    SrcVal : SDValue();
----------------
RKSimon wrote:
> Can you use APIntOps::isMask here?
> ```
> APIntOps::isMask(VT.getScalarSizeInBits(), C)
> ```
Yes! I did not see this interface before. Thank you.


Repository:
  rL LLVM

https://reviews.llvm.org/D28216





More information about the llvm-commits mailing list