inorder.py: Use insn_trace more consistently.

diff --git a/src/openpower/cyclemodel/inorder.py b/src/openpower/cyclemodel/inorder.py
index da5cbbe0df1260be6482e34dd023eedeeac144ae..c0a42ccda6b3a7e12c33226c396faaa5a9d8e713 100644 (file)
--- a/src/openpower/cyclemodel/inorder.py
+++ b/src/openpower/cyclemodel/inorder.py
@@ -69,6 +69,10 @@ def read_file(fname):
     this function is a generator, it yields a list comprising each line:
     ["insn", Hazard(...), Hazard(....), ....]
 
+    The ["insn", Hazard(...), Hazard(....), ....] format has been given the
+    name 'insn_trace', because 'insn' already refers to the assembler
+    instruction string.
+
     fname may be a *file* (an object) with a function named "read",
     in which case the Contract is that it is the *CALLER* that must
     take responsibility for the file: opening, closing, seeking.
@@ -82,10 +86,10 @@ def read_file(fname):
 
     for line in fname.readlines():
         (specs, insn) = map(str.strip, line.strip().split("#"))
-        line = [insn]
+        insn_trace = [insn]
         for spec in specs.split(" "):
-            line.append(Hazard._make(spec.split(":")))
-        yield line
+            insn_trace.append(Hazard._make(spec.split(":")))
+        yield insn_trace
     if not is_file:
         fname.close()
 
@@ -360,11 +364,11 @@ class TestTrace(unittest.TestCase):
             "r:GPR:3:0:64 r:GPR:2:0:64 w:GPR:1:0:64 # add  1, 3, 2",
         )
         f = io.StringIO("\n".join(lines))
-        lines = read_file(f)
+        insn_traces = list(read_file(f))
         basic_cpu.print_headings()
-        for trace in lines:
-            #print(trace)
-            basic_cpu.process_instructions(trace)
+        for insn_trace in insn_traces:
+            #print(insn_trace)
+            basic_cpu.process_instructions(insn_trace)
 
 def help():
     print ("-t             runs unit tests")