craig.topper added inline comments.

================
Comment at: clang/lib/Headers/mmintrin.h:1264
 {
-    return (__m64)__builtin_ia32_pcmpgtw((__v4hi)__m1, (__v4hi)__m2);
+    return (__m64)((__v4hi)__m1 > (__v4hi)__m2);
 }
----------------
jyknight wrote:
> craig.topper wrote:
> > Same here
> This is a short, which is always signed, so it should be ok as written.
Yeah. I don't know why I wrote that now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86855

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D86855: Convert __m... James Y Knight via Phabricator via cfe-commits
    • [PATCH] D86855: Conver... Craig Topper via Phabricator via cfe-commits

Reply via email to