[PATCH] D152926: [DAGCombine] Support truncated constants for fptosi.sat combining

[Dave Green via Phabricator via llvm-commits]

dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

I think this makes sense, so long as the truncate we peek through is accounted for in the constant. LGTM



================
Comment at: llvm/test/CodeGen/X86/fpclamptosat.ll:422
 
 ; FIXME: Failure to recognise the i128 is in i64 bounds.
 define i64 @stest_f64i64(double %x) {
----------------
Remove this comment now? Same below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152926