Page MenuHomePhabricator

[ARM][ParallelDSP] Convert to function pass

Authored by samparker on Fri, Jul 26, 3:02 AM.



Run across a whole function, visiting each basic block one at a time.

Diff Detail


Event Timeline

samparker created this revision.Fri, Jul 26, 3:02 AM
efriedma added inline comments.Fri, Jul 26, 11:22 AM
222 ↗(On Diff #211902)

Missing call to skipFunction?

samparker updated this revision to Diff 212295.Tue, Jul 30, 1:50 AM

Thanks. Added skipFunction and rebased.

efriedma accepted this revision.Tue, Jul 30, 12:09 PM


192 ↗(On Diff #212295)

Missing call to AU.addPreserved<ScalarEvolutionWrapperPass>(). Missing call to AU.addPreserved<GlobalsAAWrapperPass>(). But that's orthogonal, I guess; fine to address that in a followup.

This revision is now accepted and ready to land.Tue, Jul 30, 12:09 PM
Closed by commit rL367389: [ARM][ParallelDSP] Convert to function pass (authored by sam_parker, committed by ). · Explain WhyWed, Jul 31, 12:35 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jul 31, 12:35 AM

@efriedma Thanks, added Preserved before committing.

Heads-up, we've bisected a webrtc test failure down to this revision:

(But we're still analyzing things.)