[PATCH] D86786: GlobalISel: Combine out redundant sext_inreg

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 28 07:56:08 PDT 2020


arsenm created this revision.
arsenm added reviewers: paquette, aemerson, foad, aditya_nandakumar, dsanders, volkan.
Herald added subscribers: kerbowa, hiraditya, rovka, nhaehnle, jvesely.
Herald added a project: LLVM.
arsenm requested review of this revision.
Herald added a subscriber: wdng.

The scalar tests don't work yet, since computeNumSignBits apparently
doesn't handle sextload yet, and sext folds into the load first.


https://reviews.llvm.org/D86786

Files:
  llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h
  llvm/include/llvm/Target/GlobalISel/Combine.td
  llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
  llvm/test/CodeGen/AMDGPU/GlobalISel/combine-sext-inreg.mir
  llvm/test/CodeGen/AMDGPU/GlobalISel/saddsat.ll
  llvm/test/CodeGen/AMDGPU/GlobalISel/ssubsat.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86786.288611.patch
Type: text/x-patch
Size: 11600 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200828/af54398b/attachment.bin>


More information about the llvm-commits mailing list