This is an archive of the discontinued LLVM Phabricator instance.

Let the target adjust physical output- and anti-deps
ClosedPublic

Authored by frasercrmck on Apr 17 2020, 9:23 AM.

Details

Summary

This is potentially required by targets for correctness, or at least for optimization purposes.

Let me know what you think. This is necessary for our downstream target as we have a complicated per-operand scheduling model. However, as I don't think it's too controversial I thought I would push it back upstream.

Diff Detail

Event Timeline

frasercrmck created this revision.Apr 17 2020, 9:23 AM
rampitec accepted this revision.Apr 17 2020, 12:47 PM

LGTM. Since we have adjustSchedDependency() there is no reason to use it selectively.

This revision is now accepted and ready to land.Apr 17 2020, 12:47 PM
This revision was automatically updated to reflect the committed changes.