[PATCH] D26938: [X86][SSE] Improve UINT_TO_FP v2i32 -> v2f64

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 24 06:28:15 PST 2016


craig.topper added a comment.

Weird I thought I wrote a comment before. But here it is again.



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:14135
+  MVT VT = Op.getSimpleValueType();
+  MVT SrcVT = N0.getSimpleValueType();
+
----------------
SrcVT is known to be v2i32 here right? Should we just assert that that is the case and just write it explicitly in the getUNDEF call below?


Repository:
  rL LLVM

https://reviews.llvm.org/D26938





More information about the llvm-commits mailing list