This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GlobalISel: Remove unnecessary legality constraint for G_EXTRACT
ClosedPublic

Authored by tstellar on Jul 20 2018, 8:13 AM.

Details

Summary

We were marking G_EXTRACT operations unsupported if the output type
was larger than the input type. I don't see how this could ever actually
happen, so I dropped the constraint. Doing this makes it possible to
reuse the same legality code for G_INSERT.

Diff Detail

Repository
rL LLVM

Event Timeline

tstellar created this revision.Jul 20 2018, 8:13 AM
arsenm accepted this revision.Jul 20 2018, 8:17 AM

LGTM

This revision is now accepted and ready to land.Jul 20 2018, 8:17 AM
This revision was automatically updated to reflect the committed changes.