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.

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

Minor perf improvements. #1580

Merged
merged 3 commits into from
Aug 12, 2019
Merged

Commits on Aug 7, 2019

  1. Minor perf improvements.

    - Cache the vector sizes in IExecutionFrame and NodeIndexInfo to avoid calls to size().
      - 2 instructions instead of 10
    - Remove an unnecessary check in IExecutionFrame
      - add a check to the ctor so we guarantee it's unnecessary
    - Reserve memory for the vectors in BroadcastIterator
      - saves reallocs if more than one value is added
        - but rare with the mlperf models for multiple values to be added so benefit is limited.
      - slight tweak to the Broadcaster ctor code to make it more readable
    skottmckay committed Aug 7, 2019
    Configuration menu
    Copy the full SHA
    f824368 View commit details
    Browse the repository at this point in the history
  2. Fix linux build error

    skottmckay committed Aug 7, 2019
    Configuration menu
    Copy the full SHA
    57d52b7 View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2019

  1. Configuration menu
    Copy the full SHA
    13432a3 View commit details
    Browse the repository at this point in the history