craig.topper added a comment. What about rndscaless/rndscalesd?
================ Comment at: clang/lib/CodeGen/CGBuiltin.cpp:8307 + Dst = Ops[0]; + Mask = llvm::ConstantInt::get(CGF.Builder.getInt32Ty(), 1); + } else { ---------------- I'm not sure we should even try to emit a mask for the legacy scalar intrinsics. Does this get removed well by the middle or backend? ================ Comment at: clang/lib/CodeGen/CGBuiltin.cpp:8320 + Dst = Src; + Mask = llvm::ConstantInt::getAllOnesValue(CGF.Builder.getInt32Ty()); + } ---------------- Why Int32? That's not the right mask width for the legacy intrinsics. https://reviews.llvm.org/D45202 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits