[PATCH] D77931: [GlobalISel] Enable artifact combiner to combine starting from a G_MERGE_VALUES

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 10 23:09:46 PDT 2020


aemerson created this revision.
aemerson added reviewers: arsenm, paquette, dsanders, gargaroff, qcolombet.
aemerson added a project: LLVM.
Herald added subscribers: atanasyan, jrtc27, rovka, wdng, sdardis.

We generally only combine starting from users to defs in the artifact combiner, but this doesn't catch cases where at the point of combining a G_UNMERGE we don't yet have the opposite G_MERGE on input yet since we haven't legalized that far.

This change adds the users of a G_MERGE to the artifact combiner worklist if one of the uses is a G_UNMERGE.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77931

Files:
  llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-sext-zext-128.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalizer-combiner.mir
  llvm/test/CodeGen/Mips/GlobalISel/legalizer/ctlz.mir
  llvm/test/CodeGen/Mips/GlobalISel/legalizer/ctpop.mir
  llvm/test/CodeGen/Mips/GlobalISel/legalizer/cttz.mir
  llvm/test/CodeGen/Mips/GlobalISel/legalizer/zextLoad_and_sextLoad.mir
  llvm/test/CodeGen/Mips/GlobalISel/legalizer/zext_and_sext.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77931.256744.patch
Type: text/x-patch
Size: 11763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200411/b5c9cb50/attachment.bin>


More information about the llvm-commits mailing list