changeset cc41d310241f in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=cc41d310241f
description:
        Ruby Slicc: remove the call to cin.get() function
        If I understand correctly, this was put in place so that a debugger can 
be
        attached when the protocol aborts. While this sounds useful, it is a 
problem
        when the simulation is not being actively monitored. I think it is 
better to
        remove this.

diffstat:

 src/mem/slicc/ast/AST.py |  11 ++---------
 1 files changed, 2 insertions(+), 9 deletions(-)

diffs (20 lines):

diff -r 09d63f1e3559 -r cc41d310241f src/mem/slicc/ast/AST.py
--- a/src/mem/slicc/ast/AST.py  Mon Sep 10 11:57:47 2012 -0400
+++ b/src/mem/slicc/ast/AST.py  Mon Sep 10 12:20:34 2012 -0500
@@ -54,14 +54,7 @@
             message = message % args
         code = self.slicc.codeFormatter()
         code('''
-char c;
-cerr << "Runtime Error at ${{self.location}}, Ruby Time: "
-     << g_system_ptr->getTime() << ": "
-     << $message
-     << ", PID: " << getpid() << endl
-     << "press return to continue." << endl;
-cin.get(c);
-abort();
+panic("Runtime Error at ${{self.location}}, Ruby Time: %d, %s.\\n",
+    g_system_ptr->getTime(), $message);
 ''')
-
         return code
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to