Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Optimize Fence checking performance #1593

Merged
merged 2 commits into from
Aug 9, 2019
Merged

Optimize Fence checking performance #1593

merged 2 commits into from
Aug 9, 2019

Commits on Aug 8, 2019

  1. For majority of nodes, we do not need to do fence check. Instead, we … 

    …only need to do FenceCheck for CPU<->GPU mem sync node

But we pay the Fence check cost for every single node and every single input and output.

This change will minimize the Fence check to only do it when necessary.
    @ybrnathan
    ybrnathan committed Aug 8, 2019
    Configuration menu
    Copy the full SHA
    04f70c0 View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2019

  1. Fix the re-use case

    @ybrnathan
    ybrnathan committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    5967c87 View commit details
    Browse the repository at this point in the history