This is an archive of the discontinued LLVM Phabricator instance.

[LoopFlatten] Do not report CFG analyses as up-to-date
ClosedPublic

Authored by yrouban on Mar 31 2021, 9:22 PM.

Details

Summary

Re-enabling PreservedCFGChecker (D91327) the LoopFlatten was found as reporting Preserved<CFGAnalyses> while changed CFG.
This fix just removes CFGAnalyses from the returned preserved analyses set.

Diff Detail

Event Timeline

yrouban created this revision.Mar 31 2021, 9:22 PM
yrouban requested review of this revision.Mar 31 2021, 9:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2021, 9:22 PM
dmgreen accepted this revision.Apr 1 2021, 12:19 AM

Certainly sound more correct. LGTM. Thanks for the patch.

This revision is now accepted and ready to land.Apr 1 2021, 12:19 AM
TaWeiTu accepted this revision.Apr 1 2021, 12:37 AM
This revision was landed with ongoing or failed builds.Apr 1 2021, 1:53 AM
This revision was automatically updated to reflect the committed changes.