This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Remove FeatureVGPRSpilling
ClosedPublic

Authored by scott.linder on Oct 29 2018, 12:45 PM.

Details

Summary

This feature is only relevant to shaders, and is no longer used. When disabled, lowering of reserved registers for shaders causes a compiler crash.

Remove the feature and add a test for compilation of shaders at OptNone.

Diff Detail

Event Timeline

scott.linder created this revision.Oct 29 2018, 12:45 PM
arsenm accepted this revision.Oct 29 2018, 1:05 PM

LGTM

This revision is now accepted and ready to land.Oct 29 2018, 1:05 PM

Remove unused AMDGPUSubtarget::EnableVGPRSpilling; I missed this in the first patch.

arsenm accepted this revision.Oct 31 2018, 11:47 AM

LGTM

This revision was automatically updated to reflect the committed changes.
test/CodeGen/AMDGPU/schedule-vs-if-nested-loop-failure.ll