From: Bobby R. Bruce <bbruce@ucdavis.edu>
Date: Thu, 10 Sep 2020 05:26:29 +0000 (-0700)
Subject: cpu: Fixed unused var error when with fast builds
X-Git-Tag: v20.1.0.0~40
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=6e0dfe906a46ea431944357e0fdb5bda2fe57582;p=gem5.git

cpu: Fixed unused var error when with fast builds

As `is_htm_speculative` is only used in assert statements, it is
considered unused during the `.fast` compilation. This commit adds the
`M5_USED_VAR` macro.

This caused our compiler tests to fail:
https://www.mail-archive.com/gem5-dev@gem5.org/msg35913.html

Change-Id: I00d187d1a31d065c236ac29a657bd479ad4b03bc
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/34256
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Reviewed-by: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
---

diff --git a/src/cpu/simple/timing.cc b/src/cpu/simple/timing.cc
index f22c58ddd..820bede1b 100644
--- a/src/cpu/simple/timing.cc
+++ b/src/cpu/simple/timing.cc
@@ -947,7 +947,7 @@ TimingSimpleCPU::completeDataAccess(PacketPtr pkt)
     // hardware transactional memory
 
     SimpleExecContext *t_info = threadInfo[curThread];
-    const bool is_htm_speculative =
+    const bool is_htm_speculative M5_VAR_USED =
         t_info->inHtmTransactionalState();
 
     // received a response from the dcache: complete the load or store