[PATCH] D112428: [DAG] Create fptoui.sat from clamped fptosi

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 1 09:49:47 PST 2021


dmgreen updated this revision to Diff 391063.
dmgreen added a comment.

Thanks - that sounds good. I wasn't sure how the X86 converts saturated but it would be nice if they could use this same code. They were just a bit bigger in places at the moment for the unsigned cases.

This is just a rebase to the latest main, now the other parts are in tree.


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

https://reviews.llvm.org/D112428

Files:
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/lib/Target/X86/X86ISelLowering.h
  llvm/test/CodeGen/AArch64/fpclamptosat.ll
  llvm/test/CodeGen/AArch64/fpclamptosat_vec.ll
  llvm/test/CodeGen/ARM/fpclamptosat.ll
  llvm/test/CodeGen/RISCV/fpclamptosat.ll
  llvm/test/CodeGen/Thumb2/mve-fpclamptosat_vec.ll
  llvm/test/CodeGen/WebAssembly/fpclamptosat.ll
  llvm/test/CodeGen/WebAssembly/fpclamptosat_vec.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112428.391063.patch
Type: text/x-patch
Size: 60344 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211201/4d9e5418/attachment-0001.bin>


More information about the llvm-commits mailing list