[PATCH] D77210: [GlobalISel] Combine sext([sz]ext) -> [sz]ext, zext(zext) -> zext

Dominik Montada via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 04:24:53 PDT 2020


gargaroff created this revision.
gargaroff added reviewers: arsenm, volkan, aemerson, aditya_nandakumar.
Herald added subscribers: llvm-commits, kerbowa, rovka, nhaehnle, wdng, jvesely.
Herald added a project: LLVM.
gargaroff updated this revision to Diff 254163.
gargaroff added a comment.

Remove unrelated changes


Combine sext(zext x) to (zext x) since the sign-bit is 0
after the zero-extension.

Combine sext(sext x) to (sext x) and ext(zext x) to (zext x)
since the intermediate step is not needed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77210

Files:
  llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h
  llvm/test/CodeGen/AMDGPU/GlobalISel/combine-ext-legalizer.mir
  llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-saddo.mir
  llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-ssubo.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77210.254163.patch
Type: text/x-patch
Size: 8083 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200401/2622010d/attachment.bin>


More information about the llvm-commits mailing list