ARM: Get rid of the binary dumping function in utility.hh.

diff --git a/src/arch/arm/faults.cc b/src/arch/arm/faults.cc
index 3dd0b7f2f914465ec3bfcab30a300bb7b4c5ed0c..aec4df4a093cd4756d448b0938c9973e78df23f0 100644 (file)
--- a/src/arch/arm/faults.cc
+++ b/src/arch/arm/faults.cc
@@ -161,13 +161,11 @@ UndefinedInstruction::invoke(ThreadContext *tc)
 {
     assert(unknown || mnemonic != NULL);
     if (unknown) {
-        panic("Attempted to execute unknown instruction "
-              "(inst 0x%08x, opcode 0x%x, binary:%s)",
-              machInst, machInst.opcode, inst2string(machInst));
+        panic("Attempted to execute unknown instruction (inst 0x%08x)",
+              machInst);
     } else {
-        panic("Attempted to execute unimplemented instruction '%s' "
-              "(inst 0x%08x, opcode 0x%x, binary:%s)",
-              mnemonic, machInst, machInst.opcode, inst2string(machInst));
+        panic("Attempted to execute unimplemented instruction "
+                "'%s' (inst 0x%08x)", mnemonic, machInst);
     }
 }
 

diff --git a/src/arch/arm/insts/misc.cc b/src/arch/arm/insts/misc.cc
index a0af4fc2f36b3fe25cf6a479caf878e8b0b40ae1..7d383a87a4cab75221fd7fd9b98567e07b2cf62e 100644 (file)
--- a/src/arch/arm/insts/misc.cc
+++ b/src/arch/arm/insts/misc.cc
@@ -265,7 +265,5 @@ RegImmRegShiftOp::generateDisassembly(Addr pc, const SymbolTable *symtab) const
 std::string
 UnknownOp::generateDisassembly(Addr pc, const SymbolTable *symtab) const
 {
-    return csprintf("%-10s (inst 0x%x, opcode 0x%x, binary:%s)",
-                    "unknown", machInst, machInst.opcode,
-                    inst2string(machInst));
+    return csprintf("%-10s (inst %#08x)", "unknown", machInst);
 }

diff --git a/src/arch/arm/isa/formats/breakpoint.isa b/src/arch/arm/isa/formats/breakpoint.isa
index fac652d8e10fc1e50aa9e197714f2100f4d359f6..d59f6a71207dfcb54319c74fdc37d85a484e4a13 100644 (file)
--- a/src/arch/arm/isa/formats/breakpoint.isa
+++ b/src/arch/arm/isa/formats/breakpoint.isa
@@ -74,8 +74,7 @@ output decoder {{
     std::string
     Breakpoint::generateDisassembly(Addr pc, const SymbolTable *symtab) const
     {
-        return csprintf("%-10s (inst 0x%x, opcode 0x%x, binary:%s)",
-                        "Breakpoint", machInst, OPCODE, inst2string(machInst));
+        return csprintf("%-10s (inst 0x%x)", "Breakpoint", machInst);
     }
 }};
 

diff --git a/src/arch/arm/utility.hh b/src/arch/arm/utility.hh
index 7d70b31311f234dbf07538d18ad191cfa3aa1dcc..6ee2fb3257c9606b6fc2fdd64b0c983a56f1489e 100644 (file)
--- a/src/arch/arm/utility.hh
+++ b/src/arch/arm/utility.hh
@@ -147,20 +147,6 @@ namespace ArmISA {
         return (tc->readMiscRegNoEffect(MISCREG_CPSR) & 0x1f) == MODE_USER;
     }
 
-    static inline std::string
-    inst2string(MachInst machInst)
-    {
-        std::string str = "";
-        uint32_t mask = (1 << 31);
-
-        while (mask) {
-            str += ((machInst & mask) ? "1" : "0");
-            mask = mask >> 1;
-        }
-
-        return str;
-    }
-
 uint64_t getArgument(ThreadContext *tc, int number, bool fp);
     
 Fault setCp15Register(uint32_t &Rd, int CRn, int opc1, int CRm, int opc2);