This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix kernel argument alignment impacting stack size
ClosedPublic

Authored by arsenm on Jun 2 2016, 5:08 PM.

Details

Reviewers
tstellarAMD
Summary

Don't use AllocateStack because kernel arguments have nothing
to do with the stack. The ensureMaxAlignment call was still
changing the stack alignment.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 59480.Jun 2 2016, 5:08 PM
arsenm retitled this revision from to AMDGPU: Fix kernel argument alignment impacting stack size.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Jun 9 2016, 1:19 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jun 9 2016, 1:19 PM
arsenm closed this revision.Jun 17 2016, 10:24 PM

r273080