[PATCH] D86079: [X86] Improved lowering for saturating float to int.

[Bevin Hansson via Phabricator via llvm-commits]

ebevhan marked an inline comment as done.
ebevhan added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:281
 
+  if (Subtarget.hasSSE1()) {
+    // Custom lowering for saturating float to int conversions.
----------------
craig.topper wrote:
> SSE1 only supports f32 in SSE registers. f64 requires SSE2
I think this should deal with it. It checks the appropriate support in the lowering function as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86079/new/

https://reviews.llvm.org/D86079