diff options
| author | Gabe Black <gblack@eecs.umich.edu> | 2010-11-18 13:11:36 -0500 |
|---|---|---|
| committer | Gabe Black <gblack@eecs.umich.edu> | 2010-11-18 13:11:36 -0500 |
| commit | 92655b6399df526c8fe69f3b566dc9c7761782e3 (patch) | |
| tree | d66fecb0b51d81650a85201f821b053c3d1983e4 | |
| parent | 4876e0c92b7ea5d2b9a06ec652dff5a0277e12ab (diff) | |
| download | gem5-92655b6399df526c8fe69f3b566dc9c7761782e3.tar.xz | |
O3: Fix fp destination register flattening, and index offset adjusting.
This change makes O3 flatten floating point destination registers, and also
fixes misc register flattening so that it's correctly repositioned relative to
the resized regions for integer and floating point indices.
It also fixes some overly long lines.
| -rw-r--r-- | src/cpu/o3/rename_impl.hh | 32 |
1 files changed, 24 insertions, 8 deletions
diff --git a/src/cpu/o3/rename_impl.hh b/src/cpu/o3/rename_impl.hh index 4b321d099..17aef0cf5 100644 --- a/src/cpu/o3/rename_impl.hh +++ b/src/cpu/o3/rename_impl.hh @@ -959,14 +959,19 @@ DefaultRename<Impl>::renameSrcRegs(DynInstPtr &inst, ThreadID tid) RegIndex flat_src_reg = src_reg; if (src_reg < TheISA::FP_Base_DepTag) { flat_src_reg = inst->tcBase()->flattenIntIndex(src_reg); - DPRINTF(Rename, "Flattening index %d to %d.\n", (int)src_reg, (int)flat_src_reg); + DPRINTF(Rename, "Flattening index %d to %d.\n", + (int)src_reg, (int)flat_src_reg); } else if (src_reg < TheISA::Ctrl_Base_DepTag) { src_reg = src_reg - TheISA::FP_Base_DepTag; flat_src_reg = inst->tcBase()->flattenFloatIndex(src_reg); + DPRINTF(Rename, "Flattening index %d to %d.\n", + (int)src_reg, (int)flat_src_reg); flat_src_reg += TheISA::NumIntRegs; } else if (src_reg < TheISA::Max_DepTag) { - flat_src_reg = src_reg - TheISA::FP_Base_DepTag + TheISA::NumIntRegs; - DPRINTF(Rename, "Adjusting reg index from %d to %d.\n", src_reg, flat_src_reg); + flat_src_reg = src_reg - TheISA::Ctrl_Base_DepTag + + TheISA::NumFloatRegs + TheISA::NumIntRegs; + DPRINTF(Rename, "Adjusting reg index from %d to %d.\n", + src_reg, flat_src_reg); } else { panic("Reg index is out of bound: %d.", src_reg); } @@ -985,11 +990,13 @@ DefaultRename<Impl>::renameSrcRegs(DynInstPtr &inst, ThreadID tid) // See if the register is ready or not. if (scoreboard->getReg(renamed_reg) == true) { - DPRINTF(Rename, "[tid:%u]: Register %d is ready.\n", tid, renamed_reg); + DPRINTF(Rename, "[tid:%u]: Register %d is ready.\n", + tid, renamed_reg); inst->markSrcRegReady(src_idx); } else { - DPRINTF(Rename, "[tid:%u]: Register %d is not ready.\n", tid, renamed_reg); + DPRINTF(Rename, "[tid:%u]: Register %d is not ready.\n", + tid, renamed_reg); } ++renameRenameLookups; @@ -1011,12 +1018,21 @@ DefaultRename<Impl>::renameDestRegs(DynInstPtr &inst, ThreadID tid) if (dest_reg < TheISA::FP_Base_DepTag) { // Integer registers are flattened. flat_dest_reg = inst->tcBase()->flattenIntIndex(dest_reg); - DPRINTF(Rename, "Flattening index %d to %d.\n", (int)dest_reg, (int)flat_dest_reg); + DPRINTF(Rename, "Flattening index %d to %d.\n", + (int)dest_reg, (int)flat_dest_reg); + } else if (dest_reg < TheISA::Ctrl_Base_DepTag) { + dest_reg = dest_reg - TheISA::FP_Base_DepTag; + flat_dest_reg = inst->tcBase()->flattenFloatIndex(dest_reg); + DPRINTF(Rename, "Flattening index %d to %d.\n", + (int)dest_reg, (int)flat_dest_reg); + flat_dest_reg += TheISA::NumIntRegs; } else if (dest_reg < TheISA::Max_DepTag) { // Floating point and Miscellaneous registers need their indexes // adjusted to account for the expanded number of flattened int regs. - flat_dest_reg = dest_reg - TheISA::FP_Base_DepTag + TheISA::NumIntRegs; - DPRINTF(Rename, "Adjusting reg index from %d to %d.\n", dest_reg, flat_dest_reg); + flat_dest_reg = dest_reg - TheISA::Ctrl_Base_DepTag + + TheISA::NumIntRegs + TheISA::NumFloatRegs; + DPRINTF(Rename, "Adjusting reg index from %d to %d.\n", + dest_reg, flat_dest_reg); } else { panic("Reg index is out of bound: %d.", dest_reg); } |