From: Steve Reinhardt <stever@eecs.umich.edu>
Date: Tue, 4 Nov 2003 04:26:51 +0000 (-0800)
Subject: Minor changes to instruction trace output.
X-Git-Tag: m5_1.0_beta2~302
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=40b9a3878aed4e2400727ca87e1c9976cdecef58;p=gem5.git

Minor changes to instruction trace output.

arch/alpha/isa_desc:
    A few disassembly changes to make it easier to compare with old machine.def traces:
    - Make lds prefetches print f31 instead of r31 as dest.
    - Don't print mode suffixes on FP if SS_COMPATIBLE_DISASSEMBLY
cpu/exetrace.cc:
    Left-justify instruction in field, and increase width by 1.

--HG--
extra : convert_revision : 9ffd56728f1bb772aa3ccda5f027b93d4c3a4135
---

diff --git a/arch/alpha/isa_desc b/arch/alpha/isa_desc
index 75f765029..6c2888685 100644
--- a/arch/alpha/isa_desc
+++ b/arch/alpha/isa_desc
@@ -560,13 +560,19 @@ declare {{
 	{
 	    std::string mnem_str(mnemonic);
 
-	    mnem_str += ((_destRegIdx[0] >= FP_Base_DepTag)
-			 ? fpTrappingModeSuffix[trappingMode]
-			 : intTrappingModeSuffix[trappingMode]);
-	    mnem_str += roundingModeSuffix[roundingMode];
+#ifndef SS_COMPATIBLE_DISASSEMBLY
+	    std::string suffix("");
+	    suffix += ((_destRegIdx[0] >= FP_Base_DepTag)
+		       ? fpTrappingModeSuffix[trappingMode]
+		       : intTrappingModeSuffix[trappingMode]);
+	    suffix += roundingModeSuffix[roundingMode];
+
+	    if (suffix != "") {
+		mnem_str = csprintf("%s/%s", mnemonic, suffix);
+	    }
+#endif
 
 	    std::stringstream ss;
-
 	    ccprintf(ss, "%-10s ", mnem_str.c_str());
 
 	    // just print the first two source regs... if there's
@@ -1790,8 +1796,10 @@ decode OPCODE default Unknown::unknown() {
     format LoadOrPrefetch {
 	0x28: ldl({{ EA = Rb + disp; }}, {{ Ra.sl = Mem.sl; }});
 	0x29: ldq({{ EA = Rb + disp; }}, {{ Ra.uq = Mem.uq; }}, EVICT_NEXT);
+	// IsFloating flag on lds gets the prefetch to disassemble
+	// using f31 instead of r31... funcitonally it's unnecessary
 	0x22: lds({{ EA = Rb + disp; }}, {{ Fa.uq = s_to_t(Mem.ul); }},
-		  PF_EXCLUSIVE);
+		  PF_EXCLUSIVE, IsFloating);  
     }
 
     format Store {
diff --git a/cpu/exetrace.cc b/cpu/exetrace.cc
index c350288bc..ce6b0c544 100644
--- a/cpu/exetrace.cc
+++ b/cpu/exetrace.cc
@@ -74,11 +74,11 @@ Trace::InstRecord::dump(ostream &outs)
     // There's a bug in gcc 2.x library that prevents setw()
     // from working properly on strings
     string mc(staticInst->disassemble(PC, debugSymbolTable));
-    while (mc.length() < 25)
+    while (mc.length() < 26)
         mc += " ";
     outs << mc;
 #else
-    outs << setw(25) << staticInst->disassemble(PC, debugSymbolTable);
+    outs << setw(26) << left << staticInst->disassemble(PC, debugSymbolTable);
 #endif
 
     outs << " : ";