This is an archive of the discontinued LLVM Phabricator instance.

Differential D57847

Add skipFunction to PostRA machine sinking pass.
ClosedPublic

Authored by trentxintong on Feb 6 2019, 2:17 PM.

Details

Reviewers
junbuml
arsenm
Commits
rG2d84c00dfa53: Add skipFunction to PostRA machine sinking pass.
rL354541: Add skipFunction to PostRA machine sinking pass.
Summary

Add skipFunction to PostRA machine sinking pass.

Diff Detail

Event Timeline

trentxintong created this revision.Feb 6 2019, 2:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2019, 2:17 PM
Harbormaster completed remote builds in B27842: Diff 185643.Feb 6 2019, 2:17 PM
arsenm added a subscriber: arsenm.Feb 6 2019, 2:22 PM

LGTM, test would be nice though

trentxintong updated this revision to Diff 187292.Feb 18 2019, 7:32 PM

Address @arsenm comment by adding a test case. Thanks !

Herald added a subscriber: javed.absar. · View Herald TranscriptFeb 18 2019, 7:32 PM
Harbormaster completed remote builds in B28264: Diff 187292.Feb 18 2019, 7:33 PM
trentxintong added a reviewer: arsenm.Feb 20 2019, 5:58 PM
Herald added a subscriber: wdng. · View Herald TranscriptFeb 20 2019, 5:58 PM
arsenm accepted this revision.Feb 20 2019, 6:01 PM

LGTM

test/CodeGen/AArch64/bisect-post-ra-machine-sink.mir
9 ↗(On Diff #187292)

This shouldn't use label

11 ↗(On Diff #187292)

Or this

17 ↗(On Diff #187292)

Or this

19 ↗(On Diff #187292)

Or this

This revision is now accepted and ready to land.Feb 20 2019, 6:01 PM
Closed by commit rL354541: Add skipFunction to PostRA machine sinking pass. (authored by trentxintong). · Explain WhyFeb 20 2019, 6:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
lib/
CodeGen/
3 lines

Diff 185643

lib/CodeGen/MachineSink.cpp

Show First 20 Lines • Show All 1,196 Lines • ▼ Show 20 Lines for (auto I = CurBB.rbegin(), E = CurBB.rend(); I != E;) {
Changed = true; Changed = true;
++NumPostRACopySink; ++NumPostRACopySink;
} }
return Changed; return Changed;
} }
bool PostRAMachineSinking::runOnMachineFunction(MachineFunction &MF) { bool PostRAMachineSinking::runOnMachineFunction(MachineFunction &MF) {
if (skipFunction(MF.getFunction()))
return false;
bool Changed = false; bool Changed = false;
const TargetRegisterInfo *TRI = MF.getSubtarget().getRegisterInfo(); const TargetRegisterInfo *TRI = MF.getSubtarget().getRegisterInfo();
const TargetInstrInfo *TII = MF.getSubtarget().getInstrInfo(); const TargetInstrInfo *TII = MF.getSubtarget().getInstrInfo();
ModifiedRegUnits.init(*TRI); ModifiedRegUnits.init(*TRI);
UsedRegUnits.init(*TRI); UsedRegUnits.init(*TRI);
for (auto &BB : MF) for (auto &BB : MF)
Changed |= tryToSinkCopy(BB, MF, TRI, TII); Changed |= tryToSinkCopy(BB, MF, TRI, TII);
return Changed; return Changed;
} }