diff options
| author | Andreas Sandberg <andreas.sandberg@arm.com> | 2017-02-27 13:17:51 +0000 |
|---|---|---|
| committer | Andreas Sandberg <andreas.sandberg@arm.com> | 2017-03-07 11:50:35 +0000 |
| commit | b043dcf58ad766582aeab162fb855cc3fc95f2cf (patch) | |
| tree | 1da517346bef667cf20f80a54c934a3931129e4f /src/mem/ruby/system/GPUCoalescer.hh | |
| parent | c07a2d68f34ab4cef8b3ed4d079316d2b4306244 (diff) | |
| download | gem5-b043dcf58ad766582aeab162fb855cc3fc95f2cf.tar.xz | |
gpu-compute: Fix Python/C++ object hierarchy discrepancies
The GPUCoalescer and the Shader classes have different base classes in
C++ and Python. This causes subtle bugs in SWIG and compilation errors
for PyBind.
Change-Id: I1ddd2a8ea43f083470538ddfea891347b21d14d8
Reviewed-by: Andreas Hansson <andreas.hansson@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/2228
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Tony Gutierrez <anthony.gutierrez@amd.com>
Reviewed-by: Pierre-Yves Péneau <pierre-yves.peneau@lirmm.fr>
Reviewed-by: Bradford Beckmann <brad.beckmann@amd.com>
Diffstat (limited to 'src/mem/ruby/system/GPUCoalescer.hh')
| -rw-r--r-- | src/mem/ruby/system/GPUCoalescer.hh | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/mem/ruby/system/GPUCoalescer.hh b/src/mem/ruby/system/GPUCoalescer.hh index 557d39235..2b42e1933 100644 --- a/src/mem/ruby/system/GPUCoalescer.hh +++ b/src/mem/ruby/system/GPUCoalescer.hh @@ -49,7 +49,7 @@ #include "mem/request.hh" #include "mem/ruby/common/Address.hh" #include "mem/ruby/common/Consumer.hh" -#include "mem/ruby/system/RubyPort.hh" +#include "mem/ruby/system/Sequencer.hh" class DataBlock; class CacheMsg; @@ -255,10 +255,6 @@ class GPUCoalescer : public RubyPort bool handleLlsc(Addr address, GPUCoalescerRequest* request); - // Private copy constructor and assignment operator - GPUCoalescer(const GPUCoalescer& obj); - GPUCoalescer& operator=(const GPUCoalescer& obj); - class IssueEvent : public Event { private: @@ -370,6 +366,11 @@ class GPUCoalescer : public RubyPort std::vector<Stats::Histogram *> m_InitialToForwardDelayHist; std::vector<Stats::Histogram *> m_ForwardToFirstResponseDelayHist; std::vector<Stats::Histogram *> m_FirstResponseToCompletionDelayHist; + +private: + // Private copy constructor and assignment operator + GPUCoalescer(const GPUCoalescer& obj); + GPUCoalescer& operator=(const GPUCoalescer& obj); }; inline std::ostream& |