Page MenuHomePhabricator

AMDGPU: Fix recomputing dominator tree unnecessarily
ClosedPublic

Authored by arsenm on Sep 21 2015, 1:32 PM.

Details

Reviewers
tstellarAMD
Summary

SIFixSGPRCopies does not modify the CFG, but this was
being recomputed before running SIFoldOperands.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 35301.Sep 21 2015, 1:32 PM
arsenm retitled this revision from to AMDGPU: Fix recomputing dominator tree unnecessarily.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
arsenm updated this revision to Diff 35338.Sep 21 2015, 7:51 PM

Mark more passes as preserving the CFG. This saves another MachineDominatorTree run

tstellarAMD accepted this revision.Sep 25 2015, 9:55 AM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Sep 25 2015, 9:55 AM
arsenm closed this revision.Sep 25 2015, 10:23 AM

r248587