This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GlobalISel: Legalize G_GLOBAL_VALUE
ClosedPublic

Authored by arsenm on Sep 29 2019, 11:10 PM.

Details

Summary

Handle other cases besides LDS. Mostly a straight port of the existing
handling, without the intermediate custom nodes.

Diff Detail

Event Timeline

arsenm created this revision.Sep 29 2019, 11:10 PM
nhaehnle accepted this revision.Sep 30 2019, 10:12 AM

LGTM, with two nitpicks.

lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
1517–1518

There is no SDNode here... maybe: "global values are lowered".

1549–1550

Remove one empty line.

This revision is now accepted and ready to land.Sep 30 2019, 10:12 AM
arsenm closed this revision.Sep 30 2019, 6:12 PM
arsenm marked 2 inline comments as done.

r373286