This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Forward the scheduling state for PowerPC target
Needs ReviewPublic

Authored by steven.zhang on Apr 28 2019, 7:54 PM.

Details

Reviewers
fhahn
jonpa
kbarton
nemanjai
hfinkel
jsji
Group Reviewers
Restricted Project
Summary

The patch https://reviews.llvm.org/D61248 add the scheduling state forward infrastructure, and the patch https://reviews.llvm.org/D59480 add the sched state structure to keep the scheduling state. This patch bases on these two patches, and enable the scheduling state forward for PostGenericScheduler.
Then, enable it for powerpc target.

Diff Detail

Event Timeline

steven.zhang created this revision.Apr 28 2019, 7:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2019, 7:54 PM
jsji added a reviewer: jsji.Apr 29 2019, 7:26 AM
fhahn added a comment.Jun 17 2019, 3:35 AM

Would it be possible to add a test where the forwarding kicks in?

steven.zhang added a comment.EditedJun 17 2019, 11:37 PM

Would it be possible to add a test where the forwarding kicks in?

It is not easy. We could easily get some tests that have different code sequence if enable the fwd scheduling, however, it is not easy to explain why, and if that makes sense. Because, we have so many heuristic that works together to get the final order. Anyway, I will try my best to cook one, Thank you for this suggestion!

jsji added a reviewer: Restricted Project.Jan 30 2020, 7:19 AM
jsji added a project: Restricted Project.
jsji resigned from this revision.Jun 2 2022, 7:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2022, 7:42 AM
Herald added a subscriber: ecnelises. · View Herald Transcript