[PATCH] D77516: AMDGPU/GlobalISel: Work around forming illegal zextload after legalize

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 5 16:02:03 PDT 2020


arsenm created this revision.
arsenm added reviewers: dsanders, aditya_nandakumar, paquette, aemerson, kerbowa.
Herald added subscribers: hiraditya, t-tye, tpr, dstuttard, rovka, yaxunl, nhaehnle, wdng, jvesely, kzhuravl.

Selection would fail after the post legalize combiner put an illegal
zextload back together.

      

The base combiner has parameter to only allow legal operations, but
they appear to not be used. I also don't see a nice way to remove a
single entry from all_combines, so just hack around this.


https://reviews.llvm.org/D77516

Files:
  llvm/lib/Target/AMDGPU/AMDGPUCombine.td
  llvm/test/CodeGen/AMDGPU/GlobalISel/zextload.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77516.255206.patch
Type: text/x-patch
Size: 9237 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200405/c8930274/attachment.bin>


More information about the llvm-commits mailing list