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

Reply via email to