Hello Giacomo Travaglini,

I'd like you to do a code review. Please visit

    https://gem5-review.googlesource.com/9201

to review the following change.


Change subject: arch-arm: Fix unused variable warning in faults.cc
......................................................................

arch-arm: Fix unused variable warning in faults.cc

Change-Id: Ife4a2189e140cdefcf53fa88213d8a5225067457
Reviewed-by: Jack Travaglini <[email protected]>
---
M src/arch/arm/faults.cc
1 file changed, 1 insertion(+), 2 deletions(-)



diff --git a/src/arch/arm/faults.cc b/src/arch/arm/faults.cc
index a048fa8..bdb6f25 100644
--- a/src/arch/arm/faults.cc
+++ b/src/arch/arm/faults.cc
@@ -479,7 +479,6 @@
     // ARMv7 (ARM ARM issue C B1.9)

     bool have_security       = ArmSystem::haveSecurity(tc);
-    bool have_virtualization = ArmSystem::haveVirtualization(tc);

     FaultBase::invoke(tc);
     if (!FullSystem)
@@ -584,7 +583,7 @@
             setSyndrome(tc, MISCREG_HSR);
         break;
       case MODE_HYP:
-        assert(have_virtualization);
+        assert(ArmSystem::haveVirtualization(tc));
         tc->setMiscReg(MISCREG_SPSR_HYP, saved_cpsr);
         setSyndrome(tc, MISCREG_HSR);
         break;

--
To view, visit https://gem5-review.googlesource.com/9201
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Ife4a2189e140cdefcf53fa88213d8a5225067457
Gerrit-Change-Number: 9201
Gerrit-PatchSet: 1
Gerrit-Owner: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to