This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Reorder CSR reloads in epilogue to follow the same order as CSR saves in the prologue
AbandonedPublic

Authored by Yi-Hong.Lyu on May 15 2019, 7:23 PM.

Details

Reviewers
power-llvm-team
hfinkel
echristo
nemanjai
Group Reviewers
Restricted Project
Summary

This is the following patch for https://reviews.llvm.org/D61966

Diff Detail

Event Timeline

Yi-Hong.Lyu created this revision.May 15 2019, 7:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2019, 7:23 PM
jsji added a reviewer: Restricted Project.Aug 23 2019, 7:45 AM
jsji added a comment.Aug 27 2019, 7:43 PM

Are you still pursuing this Patch?

llvm/lib/Target/PowerPC/PPCFrameLowering.cpp
2336

Can you be more specific about some PPC CPUs? And how much better?
Have you tried other CPU? Any performance change for other CPU?
If this is not general better for all CPUs, have you considered at least add some options to revert to old order?

Yi-Hong.Lyu abandoned this revision.Sep 18 2019, 11:08 AM