This is an archive of the discontinued LLVM Phabricator instance.

[ScheduleDAG] Fix error assert target
ClosedPublic

Authored by clearlov7e77 on May 24 2023, 2:04 AM.

Details

Summary

[ScheduleDAG] Fix error assert target

In SUnit::removePred,N->WeakSuccsLeft is reduced but WeakSuccsLeft is checked

Diff Detail

Event Timeline

clearlov7e77 created this revision.May 24 2023, 2:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 24 2023, 2:04 AM
clearlov7e77 requested review of this revision.May 24 2023, 2:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 24 2023, 2:04 AM
clearlov7e77 retitled this revision from [ScheduleDAG] Fix error assert Target to [ScheduleDAG] Fix error assert target.May 24 2023, 2:15 AM
clearlov7e77 edited the summary of this revision. (Show Details)

LGTM, thanks!

kerbowa accepted this revision.May 24 2023, 8:56 PM
This revision is now accepted and ready to land.May 24 2023, 8:56 PM

Thanks for the review @kerbowa. I don't think I have commit access to this repository. Could you please land this patch on my behalf ?
Name - lennyxiao
email - flauti@qq.com
github username - clearlov7e77

hello @kerbowa, this is my first patch, can it be landed?Thanks for your time.