[PATCH] D70771: [NFC][PowerPC] Replace the PPCISD:: SExtVElems with ISD::SIGN_EXTEND_INREG to leverage the combine rules

qshanz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 27 00:51:23 PST 2019


steven.zhang created this revision.
steven.zhang added reviewers: nemanjai, jsji, hfinkel, PowerPC.
Herald added subscribers: shchenz, wuzish, kbarton, hiraditya.
Herald added a project: LLVM.
steven.zhang added a parent revision: D70230: [DAGCombine] Don't check the legality of type when combine the SIGN_EXTEND_INREG.

The PPCISD::SExtVElems was added by commit https://reviews.llvm.org/D34009 However, we have another ISD node ISD::SIGN_EXTEND_INREG  that perfectly match the semantics of SExtVElems. And the DAGCombiner has some combine rules for SIGN_EXTEND_INREG that produce better code.


https://reviews.llvm.org/D70771

Files:
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.h
  llvm/lib/Target/PowerPC/PPCInstrInfo.td
  llvm/lib/Target/PowerPC/PPCInstrVSX.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70771.231187.patch
Type: text/x-patch
Size: 5441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191127/d1118ff8/attachment.bin>


More information about the llvm-commits mailing list