[PATCH] D48619: [X86] Limit the number of target specific nodes emitted in LowerShiftParts

[Craig Topper via Phabricator via llvm-commits]

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

The important part is the creation of the SHLD/SHRD nodes. The compare and the conditional move can use target independent nodes that can be legalized on their own. This gives some opportunities to trigger the optimizations present in the lowering for those things. And its just better to limit the number of places we emit target specific nodes.

The changed test cases still aren't optimal.


https://reviews.llvm.org/D48619

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/legalize-shift-64.ll
  test/CodeGen/X86/pr32282.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48619.152992.patch
Type: text/x-patch
Size: 4831 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180627/7a8d42c4/attachment.bin>