Page MenuHomePhabricator
Differential D79813

[Statepoint] Mark FixupStatepointCallerSaved as preserving the CFG
ClosedPublic
Actions

Authored by craig.topper on May 12 2020, 3:19 PM.

Details

Reviewers
skatkov
nikic
Commits
rGde92dc2850c1: [Statepoint] Mark FixupStatepointCallerSaved as preserving the CFG
Summary

I can't find anything obvious in this pass that invalidates the CFG. So mark it as preserving so we stop rerunning some analysis passes.

I'm hoping this will restore some compile time lost by D75936 and D75937.

Diff Detail

Event Timeline

craig.topper created this revision.May 12 2020, 3:19 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 12 2020, 3:19 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
skatkov accepted this revision.May 12 2020, 7:29 PM

You are right, CFG is not modified. Thank you for the change.

This revision is now accepted and ready to land.May 12 2020, 7:29 PM
Closed by commit rGde92dc2850c1: [Statepoint] Mark FixupStatepointCallerSaved as preserving the CFG (authored by craig.topper). · Explain WhyMay 13 2020, 11:25 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
llvm/
lib/
CodeGen/
1 line
test/
CodeGen/
ARM/
3 lines
X86/
2 lines

Diff 263797

View Options

llvm/lib/CodeGen/FixupStatepointCallerSaved.cpp

Loading...
View Options

llvm/test/CodeGen/ARM/O3-pipeline.ll

Loading...
View Options

llvm/test/CodeGen/X86/O3-pipeline.ll

Loading...