arch-gcn3: Fixing .fast compilation for gcn3
DPRINTF was altered here: https://gem5-review.googlesource.com/c/public/gem5/+/44988. This change results in DPRINTFs always compiling. As such, the variables decladed within NDEBUG ifdefs, and later used in DPRINTFs, cause an error when compiling .fast. In this patch the NDEBUG ifdefs have been removed. Change-Id: I54992cfe152c84b265e64e1389bf2656c95ba42e Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/45481 Reviewed-by: Matthew Poremba <matthew.poremba@amd.com> Reviewed-by: Matt Sinclair <mattdsinclair@gmail.com> Maintainer: Matthew Poremba <matthew.poremba@amd.com> Maintainer: Matt Sinclair <mattdsinclair@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
@@ -960,10 +960,10 @@ namespace X86ISA
|
||||
|
||||
Process *p = sender_state->tc->getProcessPtr();
|
||||
Addr vaddr = pkt->req->getVaddr();
|
||||
#ifndef NDEBUG
|
||||
|
||||
Addr alignedVaddr = p->pTable->pageAlign(vaddr);
|
||||
assert(alignedVaddr == virtPageAddr);
|
||||
#endif
|
||||
|
||||
const EmulationPageTable::Entry *pte = p->pTable->lookup(vaddr);
|
||||
if (!pte && sender_state->tlbMode != BaseTLB::Execute &&
|
||||
p->fixupFault(vaddr)) {
|
||||
@@ -1164,10 +1164,9 @@ namespace X86ISA
|
||||
Process *p = tc->getProcessPtr();
|
||||
|
||||
Addr vaddr = pkt->req->getVaddr();
|
||||
#ifndef NDEBUG
|
||||
|
||||
Addr alignedVaddr = p->pTable->pageAlign(vaddr);
|
||||
assert(alignedVaddr == virt_page_addr);
|
||||
#endif
|
||||
|
||||
const EmulationPageTable::Entry *pte =
|
||||
p->pTable->lookup(vaddr);
|
||||
|
||||
Reference in New Issue
Block a user