From 0be64ffe2f4ff8824b3084362706ffbf456ea490 Mon Sep 17 00:00:00 2001 From: Steve Reinhardt Date: Sat, 31 May 2014 18:00:23 -0700 Subject: style: eliminate equality tests with true and false Using '== true' in a boolean expression is totally redundant, and using '== false' is pretty verbose (and arguably less readable in most cases) compared to '!'. It's somewhat of a pet peeve, perhaps, but I had some time waiting for some tests to run and decided to clean these up. Unfortunately, SLICC appears not to have the '!' operator, so I had to leave the '== false' tests in the SLICC code. --- src/arch/sparc/tlb.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/arch/sparc') diff --git a/src/arch/sparc/tlb.cc b/src/arch/sparc/tlb.cc index 72126a928..b0267718d 100644 --- a/src/arch/sparc/tlb.cc +++ b/src/arch/sparc/tlb.cc @@ -290,7 +290,7 @@ TLB::demapContext(int partition_id, int context_id) for (int x = 0; x < size; x++) { if (tlb[x].range.contextId == context_id && tlb[x].range.partitionId == partition_id) { - if (tlb[x].valid == true) { + if (tlb[x].valid) { freeList.push_front(&tlb[x]); } tlb[x].valid = false; @@ -329,7 +329,7 @@ TLB::flushAll() lookupTable.clear(); for (int x = 0; x < size; x++) { - if (tlb[x].valid == true) + if (tlb[x].valid) freeList.push_back(&tlb[x]); tlb[x].valid = false; tlb[x].used = false; -- cgit v1.2.3