This is an archive of the discontinued LLVM Phabricator instance.

Add skipFunction to PostRA machine sinking pass.
ClosedPublic

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

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
arsenm added a subscriber: arsenm.Feb 6 2019, 2:22 PM

LGTM, test would be nice though

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

arsenm accepted this revision.Feb 20 2019, 6:01 PM

LGTM

test/CodeGen/AArch64/bisect-post-ra-machine-sink.mir
9

This shouldn't use label

11

Or this

17

Or this

19

Or this

This revision is now accepted and ready to land.Feb 20 2019, 6:01 PM
This revision was automatically updated to reflect the committed changes.