commit 28bd53e4e9fb426b2531f9ffb20327d6524db339
author Nikos Nikoleris <nikos.nikoleris@arm.com> Wed Jun 13 15:48:29 2018 +0100
committer Nikos Nikoleris <nikos.nikoleris@arm.com> Wed Jun 13 17:54:52 2018 +0000
tree 20798fe251b5e1220a80a4d106d885a9e5248d41
parent 6c77e7692bd581844526401f449a69ebf0b8afe4

mem-cache: Remove unnecessary cast in SectorTags::findVictim

Removes an uneccessary cast that also caused an unused variable
error (due to -Werror) when compiling .fast targets.

Change-Id: Ic043f462925e7eaa7b691455f1d9e08a1c101980
Reviewed-on: https://gem5-review.googlesource.com/11119
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>

src/mem/cache/tags/sector_tags.cc

diff --git a/src/mem/cache/tags/sector_tags.cc b/src/mem/cache/tags/sector_tags.cc
index cec7bd4..33269ec 100644
--- a/src/mem/cache/tags/sector_tags.cc
+++ b/src/mem/cache/tags/sector_tags.cc
@@ -278,7 +278,7 @@
     }
 
     // Get the location of the victim block within the sector
-    CacheBlk* victim = victim_sector->blks[extractSectorOffset(addr)];
+    SectorSubBlk* victim = victim_sector->blks[extractSectorOffset(addr)];
 
     // Get evicted blocks. Blocks are only evicted if the sectors mismatch and
     // the currently existing sector is valid.
@@ -294,10 +294,9 @@
         }
     }
 
-    SectorSubBlk* victim_cast = static_cast<SectorSubBlk*>(victim);
     DPRINTF(CacheRepl, "set %x, way %x, sector offset %x: %s\n",
             "selecting blk for replacement\n", victim->set, victim->way,
-            victim_cast->getSectorOffset());
+            victim->getSectorOffset());
 
     return victim;
 }