[PATCH] D54640: [DAGCombiner] narrow truncated binops

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 175281.
spatel added a comment.

Patch updated:
Rebased - no code changes, but the recently added funnel-shift tests have diffs that look neutral to me.

Note that we should be limiting LEA-based regressions with D54803 <https://reviews.llvm.org/D54803>, but those are still possible, so I think it's still better to make this change in multiple steps. I have another patch drafted to help with LEA, but it depends on fixing the crash seen in D54710 <https://reviews.llvm.org/D54710>.


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

https://reviews.llvm.org/D54640

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/AMDGPU/cgp-bitfield-extract.ll
  test/CodeGen/X86/2008-09-11-CoalescerBug2.ll
  test/CodeGen/X86/2010-08-04-MaskedSignedCompare.ll
  test/CodeGen/X86/add-sub-nsw-nuw.ll
  test/CodeGen/X86/bool-math.ll
  test/CodeGen/X86/clear-lowbits.ll
  test/CodeGen/X86/cmov.ll
  test/CodeGen/X86/extract-bits.ll
  test/CodeGen/X86/extract-lowbits.ll
  test/CodeGen/X86/fshl.ll
  test/CodeGen/X86/fshr.ll
  test/CodeGen/X86/funnel-shift-rot.ll
  test/CodeGen/X86/funnel-shift.ll
  test/CodeGen/X86/pr32284.ll
  test/CodeGen/X86/pr37879.ll
  test/CodeGen/X86/rot16.ll
  test/CodeGen/X86/rotate.ll
  test/CodeGen/X86/rotate4.ll
  test/CodeGen/X86/schedule-x86-64-shld.ll
  test/CodeGen/X86/scheduler-backtracking.ll
  test/CodeGen/X86/test-shrink.ll
  test/CodeGen/X86/xchg-nofold.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54640.175281.patch
Type: text/x-patch
Size: 103234 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181126/b089e5e9/attachment-0001.bin>