Commit 3350d21
authored
Fix shared memory Table size and accesses reporting in GUPS benchmark (#60)
* Fix shared memory Table size and accesses reporting in GUPS benchmark
Based on feedback from rkarim2 in issue #56:
- For shared memory tests, Table size now shows the actual total shared memory used
- Display shared memory allocation details (bytes per block × number of blocks)
- Use correct accesses_per_elem_sh for shared memory tests
- Report size in MB for shared memory vs GB for global memory
This fixes the misleading Table size output that showed irrelevant global memory
sizes (e.g., 4.2 GB) when running shared memory tests.
* Remove comment per reviewer feedback
Removed the comment on line 497 as requested by rkarim2 in PR review.1 parent 72d62ec commit 3350d21
1 file changed
+23
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
484 | 484 | | |
485 | 485 | | |
486 | 486 | | |
487 | | - | |
488 | | - | |
489 | | - | |
490 | | - | |
491 | | - | |
492 | | - | |
493 | | - | |
494 | | - | |
| 487 | + | |
| 488 | + | |
| 489 | + | |
| 490 | + | |
| 491 | + | |
| 492 | + | |
| 493 | + | |
| 494 | + | |
| 495 | + | |
| 496 | + | |
| 497 | + | |
| 498 | + | |
| 499 | + | |
| 500 | + | |
| 501 | + | |
| 502 | + | |
| 503 | + | |
| 504 | + | |
| 505 | + | |
| 506 | + | |
| 507 | + | |
| 508 | + | |
| 509 | + | |
495 | 510 | | |
496 | 511 | | |
497 | 512 | | |
| |||
0 commit comments