[PATCH] D55855: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 18 14:44:39 PST 2018


RKSimon created this revision.
RKSimon added reviewers: craig.topper, spatel, nikic.

Now that we use the generic ISD opcodes, we can use the generic intrinsics directly as well. This fixes the poor fast-isel codegen by not expanding to an easily broken IR code sequence.

I have a sibling clang patch in progress that I will push for review in due time.

I'm also intending to deal with the signed saturation equivalents as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D55855

Files:
  lib/IR/AutoUpgrade.cpp
  test/CodeGen/X86/avx2-intrinsics-fast-isel.ll
  test/CodeGen/X86/avx512bw-intrinsics-canonical.ll
  test/CodeGen/X86/avx512bwvl-intrinsics-canonical.ll
  test/CodeGen/X86/sse2-intrinsics-fast-isel.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55855.178792.patch
Type: text/x-patch
Size: 56637 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181218/47ac8581/attachment.bin>


More information about the llvm-commits mailing list