diff options
| author | Tim Rogers <[email protected]> | 2013-02-09 14:58:14 -0800 |
|---|---|---|
| committer | Andrew Boktor <[email protected]> | 2014-08-14 13:50:04 -0700 |
| commit | e4d7bf562e8e90fa4523efc186322bff416c0506 (patch) | |
| tree | e770f274df3bc2d2002d8ec141c6e2ca865f29c1 /src/gpgpu-sim/shader.cc | |
| parent | db6ea8d4b4a2262e4d830c034a7dd220b7669ffe (diff) | |
Fixing a slew of compiler warnings
[git-p4: depot-paths = "//depot/gpgpu_sim_research/fermi/distribution/": change = 15218]
Diffstat (limited to 'src/gpgpu-sim/shader.cc')
| -rw-r--r-- | src/gpgpu-sim/shader.cc | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/gpgpu-sim/shader.cc b/src/gpgpu-sim/shader.cc index 72a1936..8573344 100644 --- a/src/gpgpu-sim/shader.cc +++ b/src/gpgpu-sim/shader.cc @@ -830,6 +830,20 @@ void TwoLevelScheduler::cycle() { break; } } + + // tgrogers - fixing a warning about unused variables at the top of this fucntion... + // This scheduler has A LOT of copied code from the LRR scheduler. + // TODO - this thing really needs to be re-written in a modular way. + // For now, to get rid of the warnings I am including the stats epilogue + // from the original scheduler function that the author of this function forgot to copy/paste + // + // issue stall statistics: + if( !valid_inst ) + m_stats->shader_cycle_distro[0]++; // idle or control hazard + else if( !ready_inst ) + m_stats->shader_cycle_distro[1]++; // waiting for RAW hazards (possibly due to memory) + else if( !issued_inst ) + m_stats->shader_cycle_distro[2]++; // pipeline stalled } void shader_core_ctx::read_operands() |