Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44885 )

Change subject: arch-gcn3,misc: Fix .fast compilation errors for GCN3_x86
......................................................................

arch-gcn3,misc: Fix .fast compilation errors for GCN3_x86

Unused variable errors occurred when compiling gem5.fast with GCC. This
patch fixes this.

Change-Id: Iaca1fb8194c2381c0a4ba5d0ea1fb5b8f2a11829
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44885
Reviewed-by: Bobby R. Bruce <[email protected]>
Reviewed-by: Matt Sinclair <[email protected]>
Reviewed-by: Matthew Poremba <[email protected]>
Maintainer: Bobby R. Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/dev/hsa/hsa_packet_processor.cc
M src/gpu-compute/gpu_compute_driver.cc
2 files changed, 2 insertions(+), 3 deletions(-)

Approvals:
  Matthew Poremba: Looks good to me, approved
  Matt Sinclair: Looks good to me, approved
  Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/dev/hsa/hsa_packet_processor.cc b/src/dev/hsa/hsa_packet_processor.cc
index cd6ef5a..eeb64da 100644
--- a/src/dev/hsa/hsa_packet_processor.cc
+++ b/src/dev/hsa/hsa_packet_processor.cc
@@ -133,8 +133,7 @@
           "%s: write of size %d to reg-offset %d (0x%x)\n",
           __FUNCTION__, pkt->getSize(), daddr, daddr);

-    int doorbellSize = gpu_device->driver()->doorbellSize();
-    assert(doorbellSize == pkt->getSize());
+    assert(gpu_device->driver()->doorbellSize() == pkt->getSize());

     uint64_t doorbell_reg(0);
     if (pkt->getSize() == 8)
diff --git a/src/gpu-compute/gpu_compute_driver.cc b/src/gpu-compute/gpu_compute_driver.cc
index 8c79be3..ac42752 100644
--- a/src/gpu-compute/gpu_compute_driver.cc
+++ b/src/gpu-compute/gpu_compute_driver.cc
@@ -582,7 +582,7 @@

             assert(isdGPU);
             assert((args->va_addr % TheISA::PageBytes) == 0);
-            Addr mmap_offset = 0;
+            M5_VAR_USED Addr mmap_offset = 0;

             Request::CacheCoherenceFlags mtype = defaultMtype;
             Addr pa_addr = 0;

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

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Iaca1fb8194c2381c0a4ba5d0ea1fb5b8f2a11829
Gerrit-Change-Number: 44885
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Matt Sinclair <[email protected]>
Gerrit-Reviewer: Matthew Poremba <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to