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