This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Add the infrastructure to forward the scheduled state between MBB
Needs ReviewPublic

Authored by qiucf on Apr 28 2019, 7:31 PM.



Forward the scheduled state from the single pred MBB, as they are belong to the the same window from pipeline view.

Diff Detail

Event Timeline

steven.zhang created this revision.Apr 28 2019, 7:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2019, 7:31 PM
lei added a subscriber: lei.Jun 12 2019, 5:16 AM
lei added inline comments.

Do you mean Return true if scheduled state is saved successfully before leaving the MBB?


Does pred mean predecessor? I think it is best to use the full word for code documentation.


Maybe this is more clear?

/// Return predecessor if MBB have only a single predecessor.
fhahn added inline comments.Jun 17 2019, 3:28 AM

IIUC this would drop the way to specify this via the scheduling policy and could change behavior for existing policies around.

It might make sense to have state forwarding imply top down here (SchedImpl->doMBBSchedRegionsTopDown() || MF.getSubtarget().forwardScheduledState())

197 ↗(On Diff #197051)

Is there a benefit of putting this setting in here, rather than in the scheduling policy, like similar settings? Unless I am missing something, I think it would be more suitable in the scheduling policy, where we already have similar options.

steven.zhang marked 4 inline comments as done.

Address reviewer's comments. Thank you!


Thank you for the comments. I will update it.


Yes, it is. I will use the full item.



197 ↗(On Diff #197051)

Sounds good.

qiucf commandeered this revision.Aug 17 2021, 7:27 PM
qiucf added a reviewer: steven.zhang.
jonpa added a comment.Sep 16 2022, 7:52 AM

please rebase

Herald added a project: Restricted Project. · View Herald TranscriptSep 16 2022, 7:52 AM