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