Page MenuHomePhabricator
Differential D42804

BreakCriticalEdges: Update PostDominatorTree
ClosedPublic
Actions

Authored by arsenm on Feb 1 2018, 9:45 AM.

Details

Reviewers
kuhar
davide

Diff Detail

Event Timeline

arsenm created this revision.Feb 1 2018, 9:45 AM
Herald added a subscriber: wdng. · View Herald TranscriptFeb 1 2018, 9:45 AM
kuhar accepted this revision.Feb 2 2018, 6:36 AM

Looks OK.

lib/Transforms/Scalar/LinearizeCFG.cpp
2607

Old comment?

This revision is now accepted and ready to land.Feb 2 2018, 6:36 AM
arsenm added inline comments.Feb 2 2018, 7:41 AM
lib/Transforms/Scalar/LinearizeCFG.cpp
2607

This pass doesn't even exist upstream yet, so this shouldn't be in the patch at all

arsenm closed this revision.Feb 22 2019, 7:01 AM
arsenm marked 2 inline comments as done.

r354673

Revision Contents
Changeset List

PathSize
include/
llvm/
Analysis/
2 lines
Transforms/
Utils/
7 lines
lib/
Transforms/
Scalar/
2 lines
Utils/
17 lines
unittests/
Transforms/
Utils/
29 lines

Diff 132413

View Options

include/llvm/Analysis/PostDominators.h

Loading...
View Options

include/llvm/Transforms/Utils/BasicBlockUtils.h

Loading...
View Options

lib/Transforms/Scalar/LinearizeCFG.cpp

Loading...
View Options

lib/Transforms/Utils/BreakCriticalEdges.cpp

Loading...
View Options

unittests/Transforms/Utils/BasicBlockUtils.cpp

Loading...