diff options
| author | Gabe Black <gblack@eecs.umich.edu> | 2013-01-04 19:00:45 -0600 |
|---|---|---|
| committer | Gabe Black <gblack@eecs.umich.edu> | 2013-01-04 19:00:45 -0600 |
| commit | e17c375ddd32fbbef55a96c446a4b98b20df2ad5 (patch) | |
| tree | 42e917087dc0887f0aed3d74bad6196f27ab0d0c /src/cpu/inorder | |
| parent | d1965af22045d2a62b1cd1bc473b836413d79b46 (diff) | |
| download | gem5-e17c375ddd32fbbef55a96c446a4b98b20df2ad5.tar.xz | |
Decoder: Remove the thread context get/set from the decoder.
This interface is no longer used, and getting rid of it simplifies the
decoders and code that sets up the decoders. The thread context had been used
to read architectural state which was used to contextualize the instruction
memory as it came in. That was changed so that the state is now sent to the
decoders to keep locally if/when it changes. That's significantly more
efficient.
Committed by: Nilay Vaish <nilay@cs.wisc.edu>
Diffstat (limited to 'src/cpu/inorder')
| -rw-r--r-- | src/cpu/inorder/resources/fetch_unit.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/cpu/inorder/resources/fetch_unit.cc b/src/cpu/inorder/resources/fetch_unit.cc index 0ed59fe2d..49bd0434b 100644 --- a/src/cpu/inorder/resources/fetch_unit.cc +++ b/src/cpu/inorder/resources/fetch_unit.cc @@ -59,7 +59,7 @@ FetchUnit::FetchUnit(string res_name, int res_id, int res_width, instSize(sizeof(TheISA::MachInst)), fetchBuffSize(params->fetchBuffSize) { for (int tid = 0; tid < MaxThreads; tid++) - decoder[tid] = new Decoder(NULL); + decoder[tid] = new Decoder; } FetchUnit::~FetchUnit() @@ -109,7 +109,6 @@ FetchUnit::createMachInst(std::list<FetchBlock*>::iterator fetch_it, MachInst mach_inst = TheISA::gtoh(fetchInsts[fetch_offset]); - decoder[tid]->setTC(cpu->thread[tid]->getTC()); decoder[tid]->moreBytes(instPC, inst->instAddr(), mach_inst); assert(decoder[tid]->instReady()); inst->setStaticInst(decoder[tid]->decode(instPC)); |