misc: string.join has been removed in python3

In general string methods are deprecated in favour of str ones

Change-Id: Ifba04e0b70be29e5a82a67cf11837f740de57e32
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26244
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Tested-by: kokoro <noreply+kokoro@google.com>

diff --git a/src/arch/arm/isa/insts/mem.isa b/src/arch/arm/isa/insts/mem.isa
index 14a3267a09d8cbd3b7a2446109a542be7d78f094..0087a24dc1f3742caf73fe4db10c719f21e9659e 100644 (file)
--- a/src/arch/arm/isa/insts/mem.isa
+++ b/src/arch/arm/isa/insts/mem.isa
@@ -57,7 +57,7 @@ let {{
             # This shouldn't be part of the eaCode, but until the exec templates
             # are converted over it's the easiest place to put it.
             eaCode += '\n    unsigned memAccessFlags = '
-            eaCode += (string.join(memFlags, '|') + ';')
+            eaCode += ('|'.join(memFlags) + ';')
 
             codeBlobs["ea_code"] = eaCode
 

diff --git a/src/arch/isa_parser.py b/src/arch/isa_parser.py
index 8d0fee22ded16128f75d75c6d9a2caee7efba4ea..49b3b072985c00262f6dab5ff2ed5ddd0db9fff8 100755 (executable)
--- a/src/arch/isa_parser.py
+++ b/src/arch/isa_parser.py
@@ -41,7 +41,6 @@ from __future__ import with_statement, print_function
 import os
 import sys
 import re
-import string
 import inspect, traceback
 # get type names
 from types import *
@@ -231,7 +230,7 @@ class Format(object):
         self.params = params
         label = 'def format ' + id
         self.user_code = compile(fixPythonIndentation(code), label, 'exec')
-        param_list = string.join(params, ", ")
+        param_list = ", ".join(params)
         f = '''def defInst(_code, _context, %s):
                 my_locals = vars().copy()
                 exec _code in _context, my_locals
@@ -1399,7 +1398,7 @@ def makeFlagConstructor(flag_list):
             i += 1
     pre = '\n\tflags['
     post = '] = true;'
-    code = pre + string.join(flag_list, post + pre) + post
+    code = pre + (post + pre).join(flag_list) + post
     return code
 
 # Assume all instruction flags are of the form 'IsFoo'
@@ -1584,7 +1583,7 @@ class ISAParser(Grammar):
         # file where it was included.
         self.fileNameStack = Stack()
 
-        symbols = ('makeList', 're', 'string')
+        symbols = ('makeList', 're')
         self.exportContext = dict([(s, eval(s)) for s in symbols])
 
         self.maxInstSrcRegs = 0
@@ -2597,7 +2596,7 @@ StaticInstPtr
         (?<!\w)      # neg. lookbehind assertion: prevent partial matches
         ((%s)(?:_(%s))?)   # match: operand with optional '_' then suffix
         (?!\w)       # neg. lookahead assertion: prevent partial matches
-        ''' % (string.join(operands, '|'), string.join(extensions, '|'))
+        ''' % ('|'.join(operands), '|'.join(extensions))
 
         self.operandsRE = re.compile(operandsREString, re.MULTILINE|re.VERBOSE)
 
@@ -2605,7 +2604,7 @@ StaticInstPtr
         # groups are returned (base and ext, not full name as above).
         # Used for subtituting '_' for '.' to make C++ identifiers.
         operandsWithExtREString = r'(?<!\w)(%s)_(%s)(?!\w)' \
-            % (string.join(operands, '|'), string.join(extensions, '|'))
+            % ('|'.join(operands), '|'.join(extensions))
 
         self.operandsWithExtRE = \
             re.compile(operandsWithExtREString, re.MULTILINE)

diff --git a/src/arch/micro_asm.py b/src/arch/micro_asm.py
index 2745004ee1fbc988804475944116761aaf25e8df..536f70994a9fc7775257c3ccf843cb1cb40f595e 100644 (file)
--- a/src/arch/micro_asm.py
+++ b/src/arch/micro_asm.py
@@ -29,7 +29,6 @@ from __future__ import print_function
 import os
 import sys
 import re
-import string
 import traceback
 # get type names
 from types import *

diff --git a/src/arch/mips/isa/formats/util.isa b/src/arch/mips/isa/formats/util.isa
index 5e6918b5721b1b5d7823fe729a85b76867cb22c5..39ece7a2db9753c13449b7bdb5734f2986b8dd15 100644 (file)
--- a/src/arch/mips/isa/formats/util.isa
+++ b/src/arch/mips/isa/formats/util.isa
@@ -50,7 +50,7 @@ def LoadStoreBase(name, Name, ea_code, memacc_code, mem_flags, inst_flags,
 
     if mem_flags:
         mem_flags = [ 'Request::%s' % flag for flag in mem_flags ]
-        s = '\n\tmemAccessFlags = ' + string.join(mem_flags, '|') + ';'
+        s = '\n\tmemAccessFlags = ' + '|'.join(mem_flags) + ';'
         iop.constructor += s
 
     # select templates

diff --git a/src/arch/power/isa/formats/util.isa b/src/arch/power/isa/formats/util.isa
index 582e62f6f3fdc7f0786f393d413df784403de43b..1bb78e6ad2fc8d6546a49cc6078695c0f4fe0662 100644 (file)
--- a/src/arch/power/isa/formats/util.isa
+++ b/src/arch/power/isa/formats/util.isa
@@ -100,7 +100,7 @@ def LoadStoreBase(name, Name, ea_code, memacc_code, mem_flags, inst_flags,
                         inst_flags)
 
     if mem_flags:
-        s = '\n\tmemAccessFlags = ' + string.join(mem_flags, '|') + ';'
+        s = '\n\tmemAccessFlags = ' + '|'.join(mem_flags) + ';'
         iop.constructor += s
 
     fullExecTemplate = eval(exec_template_base + 'Execute')

diff --git a/src/arch/riscv/isa/formats/mem.isa b/src/arch/riscv/isa/formats/mem.isa
index 175eff999e0ec43ef89e8684f9bb496f265b5b17..faaae6f772620480987e219c5fda0de81879758d 100644 (file)
--- a/src/arch/riscv/isa/formats/mem.isa
+++ b/src/arch/riscv/isa/formats/mem.isa
@@ -73,7 +73,7 @@ def LoadStoreBase(name, Name, offset_code, ea_code, memacc_code, mem_flags,
 
     if mem_flags:
         mem_flags = [ 'Request::%s' % flag for flag in mem_flags ]
-        s = '\n\tmemAccessFlags = ' + string.join(mem_flags, '|') + ';'
+        s = '\n\tmemAccessFlags = ' + '|'.join(mem_flags) + ';'
         iop.constructor += s
 
     # select templates

diff --git a/src/arch/sparc/isa/formats/mem/swap.isa b/src/arch/sparc/isa/formats/mem/swap.isa
index 7a287066b351d075ae7b7985263098e6f9bbfa1a..ef13385bdf201ee87fa3a2400c3e37a38b242ca1 100644 (file)
--- a/src/arch/sparc/isa/formats/mem/swap.isa
+++ b/src/arch/sparc/isa/formats/mem/swap.isa
@@ -124,7 +124,7 @@ let {{
 def format Swap(code, postacc_code, mem_flags, *opt_flags) {{
     mem_flags = makeList(mem_flags)
     mem_flags = [ 'Request::%s' % flag for flag in mem_flags ]
-    flags = string.join(mem_flags, '|')
+    flags = '|'.join(mem_flags)
 
     (header_output,
      decoder_output,
@@ -137,7 +137,7 @@ def format SwapAlt(code, postacc_code, mem_flags, *opt_flags) {{
     mem_flags = makeList(mem_flags)
     mem_flags = [ 'Request::%s' % flag for flag in mem_flags ]
     mem_flags.append("EXT_ASI")
-    flags = string.join(mem_flags, '|')
+    flags = '|'.join(mem_flags)
     (header_output,
      decoder_output,
      exec_output,
@@ -169,7 +169,7 @@ def format CasAlt(code, postacc_code, mem_flags, *opt_flags) {{
     mem_flags = makeList(mem_flags)
     mem_flags = [ 'Request::%s' % flag for flag in mem_flags ]
     mem_flags.append("EXT_ASI")
-    flags = string.join(mem_flags, '|')
+    flags = '|'.join(mem_flags)
     (header_output,
      decoder_output,
      exec_output,

diff --git a/src/python/m5/util/code_formatter.py b/src/python/m5/util/code_formatter.py
index 9870430d639de736439d2830fbb1f146e970b438..f441df0af507843a62b6098da6ccb0280dca4d60 100644 (file)
--- a/src/python/m5/util/code_formatter.py
+++ b/src/python/m5/util/code_formatter.py
@@ -35,7 +35,6 @@ except ImportError:
 import inspect
 import os
 import re
-import string
 
 class lookup(object):
     def __init__(self, formatter, frame, *args, **kwargs):
@@ -164,7 +163,7 @@ class code_formatter(object):
         f.close()
 
     def __str__(self):
-        data = string.join(self._data, '')
+        data = ''.join(self._data)
         self._data = [ data ]
         return data