Matthew Poremba has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/47103 )

Change subject: arch-vega: Add missing functions referenced by insts
......................................................................

arch-vega: Add missing functions referenced by insts

Some instructions were referencing pc() and isExecMaskRegister() which
were not defined.

Change-Id: Ic5b3fa9057950ff85603fcb87447a81b6c7f274b
---
M src/arch/amdgpu/vega/insts/gpu_static_inst.hh
M src/gpu-compute/gpu_dyn_inst.hh
2 files changed, 10 insertions(+), 0 deletions(-)



diff --git a/src/arch/amdgpu/vega/insts/gpu_static_inst.hh b/src/arch/amdgpu/vega/insts/gpu_static_inst.hh
index 8d31672..28328d6 100644
--- a/src/arch/amdgpu/vega/insts/gpu_static_inst.hh
+++ b/src/arch/amdgpu/vega/insts/gpu_static_inst.hh
@@ -57,6 +57,12 @@
             return isFlatScratchReg(opIdx);
         }

+        bool
+        isExecMaskRegister(int opIdx) override
+        {
+            return isExecMask(opIdx);
+        }
+
         void initOperandInfo() override { return; }
         int getOperandSize(int opIdx) override { return 0; }

diff --git a/src/gpu-compute/gpu_dyn_inst.hh b/src/gpu-compute/gpu_dyn_inst.hh
index ab7ccb8..b82a47c 100644
--- a/src/gpu-compute/gpu_dyn_inst.hh
+++ b/src/gpu-compute/gpu_dyn_inst.hh
@@ -159,6 +159,9 @@

     InstSeqNum seqNum() const;

+    Addr pc() const { return _pc; }
+    void pc(Addr pc) { _pc = pc; }
+
     enums::StorageClassType executedAs();

     // virtual address for scalar memory operations
@@ -476,6 +479,7 @@
     const InstSeqNum _seqNum;
     int maxSrcVecRegOpSize;
     int maxSrcScalarRegOpSize;
+    Addr _pc;

     // the time the request was started
     Tick accessTime = -1;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47103
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: Ic5b3fa9057950ff85603fcb87447a81b6c7f274b
Gerrit-Change-Number: 47103
Gerrit-PatchSet: 1
Gerrit-Owner: Matthew Poremba <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
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