mike.dvoretsky added inline comments.

================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:8307
+    Dst = Ops[0];
+    Mask = llvm::ConstantInt::get(CGF.Builder.getInt32Ty(), 1);
+  } else {
----------------
craig.topper wrote:
> 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?
The masking is done to represent all operations handled here in a uniform way. 
D45203 removes it in the backend.


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