diff --git a/src/mem/cache/cache_impl.hh b/src/mem/cache/cache_impl.hh
index 9fb5cdbdeb..c8c1a239c8 100644
--- a/src/mem/cache/cache_impl.hh
+++ b/src/mem/cache/cache_impl.hh
@@ -821,7 +821,9 @@ Cache<TagStore>::handleFill(PacketPtr pkt, BlkType *blk,
                             PacketList &writebacks)
 {
     Addr addr = pkt->getAddr();
+#if TRACING_ON
     CacheBlk::State old_state = blk ? blk->status : 0;
+#endif
 
     if (blk == NULL) {
         // better have read new data...
diff --git a/src/mem/cache/miss/mshr_queue.cc b/src/mem/cache/miss/mshr_queue.cc
index 56ec62a7d0..4d3cf30e12 100644
--- a/src/mem/cache/miss/mshr_queue.cc
+++ b/src/mem/cache/miss/mshr_queue.cc
@@ -123,6 +123,7 @@ MSHRQueue::addToReadyList(MSHR *mshr)
         }
     }
     assert(false);
+    return end;  // keep stupid compilers happy
 }
 
 
diff --git a/src/mem/cache/tags/split.cc b/src/mem/cache/tags/split.cc
index e22ccbb962..ae284766d3 100644
--- a/src/mem/cache/tags/split.cc
+++ b/src/mem/cache/tags/split.cc
@@ -300,7 +300,7 @@ Split::findBlock(Addr addr) const
 SplitBlk*
 Split::findReplacement(Addr addr, PacketList &writebacks)
 {
-    SplitBlk *blk;
+    SplitBlk *blk = NULL;
 
     assert(0);
 #if 0