Remove runtime aerialvision overhead w/ preprocessor #37
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Despite the AerialVision logging not being enabled in a GPGPU-Sim config, you still pay the price of runtime overhead for collecting memory system stats on each window. This commit seeks to remove most of this overhead by adding GPGPU-Sim build mode (
av_enabled
). Whenav_enabled
is not enabled, the preprocessor removes much of the code related to AerialVision in the caches.Below are examples of where the overhead of AerialVision is paid even when it is not enabled:
gpu-cache.cc
operator+=
cache_stats_pw
is updatedThis lead to an observed 3-5% increase in observed performance.