This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix FP restore from being reordered with stack ops
ClosedPublic

Authored by arsenm on Mar 23 2018, 3:29 PM.

Details

Reviewers
rampitec
Summary

In a function, s5 is used as the frame base SGPR. Before it was
possible if there were stack operations immediately after
the call to be moved before the FP restore copy after the call.

I'm not 100% satisfied with the solution of adding an implicit
use to the ADJCALLSTACK* pseudo, but I'm not sure what would
be better.

Diff Detail

Event Timeline

arsenm created this revision.Mar 23 2018, 3:29 PM
This revision is now accepted and ready to land.Mar 23 2018, 3:52 PM
arsenm closed this revision.Mar 27 2018, 11:42 AM

r328650