Page MenuHomePhabricator

[VE] Support STACKSAVE and STACKRESTORE
ClosedPublic

Authored by kaz7 on Dec 18 2020, 6:22 AM.

Details

Summary

Change to use default expanded code. Add regression tests also.

Diff Detail

Event Timeline

kaz7 created this revision.Dec 18 2020, 6:22 AM
kaz7 requested review of this revision.Dec 18 2020, 6:22 AM
simoll added inline comments.Dec 18 2020, 9:15 AM
llvm/test/CodeGen/VE/Scalar/stacksave.ll
18–19

Why do we want %s11 to be assigned twice?

kaz7 added a comment.Dec 18 2020, 4:29 PM

Thank you for comment.

llvm/test/CodeGen/VE/Scalar/stacksave.ll
18–19

stackrestore assignes %s0 to %s11 (=%sp). Then, epilogue of this function restore %s11 again using %s9 (=%fp=saved stack pointer).

simoll accepted this revision.Dec 21 2020, 1:34 AM

Thx!

This revision is now accepted and ready to land.Dec 21 2020, 1:34 AM
This revision was landed with ongoing or failed builds.Dec 21 2020, 3:16 AM
This revision was automatically updated to reflect the committed changes.