This is an archive of the discontinued LLVM Phabricator instance.

Correctly report Changed status in FoldBranchToCommonDest
ClosedPublic

Authored by serge-sans-paille on Jun 29 2020, 3:46 AM.

Details

Summary

It's possible for the first loop trip to set the Changed Status, and to another
one to early exit, so take that into account.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJun 29 2020, 3:46 AM
foad accepted this revision.Jun 29 2020, 6:47 AM

Seems obviously correct.

This revision is now accepted and ready to land.Jun 29 2020, 6:47 AM
This revision was automatically updated to reflect the committed changes.