[NFC][Py Reformat] Reformat python files in llvm-test-suite

This is an ongoing series of commits that are reformatting our Python
code.

Reformatting is done with `black`.

If you end up having problems merging this commit because you have made
changes to a python file, the best way to handle that is to run git
checkout --ours <yourfile> and then reformat it with black.

If you run into any problems, post to discourse about it and we will try
to help.

RFC Thread below:

https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style

Specific to this change:
* Does not change third-party code in `MicroBenchmarks/libs/benchmark`.

Fix code to make it parseable by python3/black in the first place:
* Fixed space/tab confusion in `CollectDebugInfoUsingLLDB.py`.
* Change to python3 `print` syntax: `ABI-Testsuite/sample.py`,
  `ABI-Testsuite/test/lit.site.cfg`, `tools/get-report-time`

Differential Revision: https://reviews.llvm.org/D151914

Author: MatzeB

ABI-Testsuite/linux-x86.py

@@ -4,116 +4,122 @@
 from __future__ import print_function
 import os
 import sys
+
 wopt = " -w"
 test_params = {
-    "bindump" : "nm",
-    "runtool" : "", # if None, then runtool_func must exists
-    "Platform" : "linux-gcc",
-    "cxx_rtti" : "-frtti",   # option to enable rtti
-    "cxx_exceptions" : "-fexceptions",   # option to enable rtti
-    "cxx_cpp11" : "-std=c++0x" ,
+    "bindump": "nm",
+    "runtool": "",  # if None, then runtool_func must exists
+    "Platform": "linux-gcc",
+    "cxx_rtti": "-frtti",  # option to enable rtti
+    "cxx_exceptions": "-fexceptions",  # option to enable rtti
+    "cxx_cpp11": "-std=c++0x",
 }
 
+
 def build_cxx_compiler_name(cn):
-  # build c++ compiler name from C compiler name
-  # <prefix>clang <options> : becomes <prefix>clang -x c++ <options>
-  # <prefix>gcc <options> : becomes <prefix>g++ <options>
-  # <prefix>gcc-<vers> <options>: becomes <prefix>g++-<vers> <options>
+    # build c++ compiler name from C compiler name
+    # <prefix>clang <options> : becomes <prefix>clang -x c++ <options>
+    # <prefix>gcc <options> : becomes <prefix>g++ <options>
+    # <prefix>gcc-<vers> <options>: becomes <prefix>g++-<vers> <options>
 
-  cnlen = len(cn)
-  if (cn == 'clang'): #plain clang
-    return "clang -x c++"
+    cnlen = len(cn)
+    if cn == "clang":  # plain clang
+        return "clang -x c++"
 
-  idx = cn.find("clang ")
-  if (idx >= 0): # clang with some options
-    return cn[:(idx+6)] + "-x c++ " + cn[(idx+6):]
+    idx = cn.find("clang ")
+    if idx >= 0:  # clang with some options
+        return cn[: (idx + 6)] + "-x c++ " + cn[(idx + 6) :]
 
-  idx = cn.rfind('clang')
-  if (idx == (cnlen-5)): # ends in gcc
-    return cn + " -x c++"
+    idx = cn.rfind("clang")
+    if idx == (cnlen - 5):  # ends in gcc
+        return cn + " -x c++"
 
-  if (cn == 'gcc'):
-    return "g++"
+    if cn == "gcc":
+        return "g++"
 
-  idx = cn.rfind('gcc')
-  if (idx == (cnlen-3)): # ends in gcc
-    return cn + " -x c++"
+    idx = cn.rfind("gcc")
+    if idx == (cnlen - 3):  # ends in gcc
+        return cn + " -x c++"
 
-  idx = cn.find("gcc ")
-  if (idx >= 0): # gcc with some options
-    return cn[:(idx+1)] + "++ " + cn[(idx+4):]
+    idx = cn.find("gcc ")
+    if idx >= 0:  # gcc with some options
+        return cn[: (idx + 1)] + "++ " + cn[(idx + 4) :]
 
-  idx = cn.find("gcc-")
-  if (idx >= 0): # gcc-<version>
-    return cn[:(idx+1)] + "++" + cn[(idx+3):]
+    idx = cn.find("gcc-")
+    if idx >= 0:  # gcc-<version>
+        return cn[: (idx + 1)] + "++" + cn[(idx + 3) :]
+
+    return "UNEXPECTED COMPILER"
 
-  return "UNEXPECTED COMPILER"
 
 def build_linker_name(cn):
-  # build linkercompiler name from C compiler name
-  # <prefix>clang <options> : becomes <prefix>clang++ <options>
-  # <prefix>gcc <options> : becomes <prefix>g++ <options>
-  # <prefix>gcc-<vers> <options>: becomes <prefix>g++-<vers> <options>
+    # build linkercompiler name from C compiler name
+    # <prefix>clang <options> : becomes <prefix>clang++ <options>
+    # <prefix>gcc <options> : becomes <prefix>g++ <options>
+    # <prefix>gcc-<vers> <options>: becomes <prefix>g++-<vers> <options>
 
-  if (cn == 'clang'): #plain clang
-    return "clang++"
+    if cn == "clang":  # plain clang
+        return "clang++"
 
-  idx = cn.find("clang ")
-  if (idx >= 0): # clang with some options
-    return cn[:(idx+5)] + "++ " + cn[(idx+6):]
+    idx = cn.find("clang ")
+    if idx >= 0:  # clang with some options
+        return cn[: (idx + 5)] + "++ " + cn[(idx + 6) :]
 
-  cnlen = len(cn)
-  idx = cn.rfind('clang')
-  if (idx == (cnlen-5)): # ends in clang
-    return cn + "++"
+    cnlen = len(cn)
+    idx = cn.rfind("clang")
+    if idx == (cnlen - 5):  # ends in clang
+        return cn + "++"
 
-  return build_cxx_compiler_name(cn) # for gcc, linekr name is same as c++ compiler name
+    return build_cxx_compiler_name(
+        cn
+    )  # for gcc, linekr name is same as c++ compiler name
 
 
 import subprocess
+
 if __name__ == "__main__":
-  print(sys.argv)
-  litargs = [sys.argv[0]] # args for lit.main
-  c_compiler_name = sys.argv[1].strip()
-  cxx_compiler_name = build_cxx_compiler_name(c_compiler_name)
-  linker_name = build_linker_name(c_compiler_name)
-  #print("C++ compiler:", cxx_compiler_name)
-  #print("linker :", linker_name)
-  for la in sys.argv[2:]:
-    litargs.append(la)
-  sys.argv = litargs
-  cmd = c_compiler_name.split() + [ "-o", "b.exe", "test/common/genselector.c"]
-  #print("cmd = ", cmd)
-  rv = subprocess.call(cmd)
-  #print("rv = " , rv)
-  if rv != 0:
-    print("compiler failed: ", cmd)
-    sys.exit(1)
-  rv = subprocess.call("./b.exe")
-  fp = open("nselector.h")
-  fi = fp.read();
-  fp.close()
-  if ("LP64" in fi):
-    test_params["Mode"] = "LP64-x86"
-    prf = "--check-prefix=LP64"
-  else:
-    # 'gcc -m32' or 'clang -m32' mode
-    test_params["Mode"] = "ILP32-x86"
-    prf = "--check-prefix=ILP32"
-  test_params["prefixes"] = prf;
-  test_params["c_compiler"] = c_compiler_name
-  test_params["cxx_compiler"] = cxx_compiler_name
-  test_params["linker"] = linker_name
-  test_params["Platform"] = "linux-" + cmd[0]
-
-  print("test_params = " , test_params)
-  builtin_parameters = {
-    'build_mode' : "Release",
-    'llvm_site_config' : os.path.join(os.getcwd(), 'lit.site.cfg'),
-    'clang_site_config': os.path.join(os.getcwd(), 'lit.site.cfg'),
-    'test_params' : test_params
+    print(sys.argv)
+    litargs = [sys.argv[0]]  # args for lit.main
+    c_compiler_name = sys.argv[1].strip()
+    cxx_compiler_name = build_cxx_compiler_name(c_compiler_name)
+    linker_name = build_linker_name(c_compiler_name)
+    # print("C++ compiler:", cxx_compiler_name)
+    # print("linker :", linker_name)
+    for la in sys.argv[2:]:
+        litargs.append(la)
+    sys.argv = litargs
+    cmd = c_compiler_name.split() + ["-o", "b.exe", "test/common/genselector.c"]
+    # print("cmd = ", cmd)
+    rv = subprocess.call(cmd)
+    # print("rv = " , rv)
+    if rv != 0:
+        print("compiler failed: ", cmd)
+        sys.exit(1)
+    rv = subprocess.call("./b.exe")
+    fp = open("nselector.h")
+    fi = fp.read()
+    fp.close()
+    if "LP64" in fi:
+        test_params["Mode"] = "LP64-x86"
+        prf = "--check-prefix=LP64"
+    else:
+        # 'gcc -m32' or 'clang -m32' mode
+        test_params["Mode"] = "ILP32-x86"
+        prf = "--check-prefix=ILP32"
+    test_params["prefixes"] = prf
+    test_params["c_compiler"] = c_compiler_name
+    test_params["cxx_compiler"] = cxx_compiler_name
+    test_params["linker"] = linker_name
+    test_params["Platform"] = "linux-" + cmd[0]
+
+    print("test_params = ", test_params)
+    builtin_parameters = {
+        "build_mode": "Release",
+        "llvm_site_config": os.path.join(os.getcwd(), "lit.site.cfg"),
+        "clang_site_config": os.path.join(os.getcwd(), "lit.site.cfg"),
+        "test_params": test_params,
     }
-  #print("builtin_params = " , test_params)
-  from lit.main import main
-  main(builtin_parameters)
+    # print("builtin_params = " , test_params)
+    from lit.main import main
 
+    main(builtin_parameters)

ABI-Testsuite/sample.py

@@ -6,36 +6,38 @@
 
 import os
 import sys
-wopt = ""   # set to " -w", or equivalent option, to disable warnings.
-
-#skiplist = [ "test/s2_7/test_02", "test/s2_7/test_06"] # Enable to choose a 'skip list'
-
-test_params =  {
-    'bindump': 'nm',                 # name of the nm-like tool to print symbol names
-    'cxx_rtti': '-frtti',            # option to enable rtti
-    'runtool': '',                   # leave empty for native runs. 
-                                     # Set to the name of the 'runner' tool for cross tools
-    'linker': 'clang++ -w',          # name of the linker
-    'cxx_compiler': 'clang -x c++'+wopt, # name of the C++ compiler
-    'c_compiler': 'clang'+wopt,      # name of the C compiler
-    'cxx_cpp11': '-std=c++0x',       # option to enable C++11 mode, if any 
-    'prefixes': '--check-prefix=LP64', # must be LP64 or ILP32
-    'Platform': 'linux-clang',       # just for documentation
-    'Mode': 'LP64-x86',              # must be "LP64-x86" or "ILP32-x86"
-    'cxx_exceptions': '-fexceptions' # options to enable exceptions
+
+wopt = ""  # set to " -w", or equivalent option, to disable warnings.
+
+# skiplist = [ "test/s2_7/test_02", "test/s2_7/test_06"] # Enable to choose a 'skip list'
+
+test_params = {
+    "bindump": "nm",  # name of the nm-like tool to print symbol names
+    "cxx_rtti": "-frtti",  # option to enable rtti
+    "runtool": "",  # leave empty for native runs.
+    # Set to the name of the 'runner' tool for cross tools
+    "linker": "clang++ -w",  # name of the linker
+    "cxx_compiler": "clang -x c++" + wopt,  # name of the C++ compiler
+    "c_compiler": "clang" + wopt,  # name of the C compiler
+    "cxx_cpp11": "-std=c++0x",  # option to enable C++11 mode, if any
+    "prefixes": "--check-prefix=LP64",  # must be LP64 or ILP32
+    "Platform": "linux-clang",  # just for documentation
+    "Mode": "LP64-x86",  # must be "LP64-x86" or "ILP32-x86"
+    "cxx_exceptions": "-fexceptions"  # options to enable exceptions
     # , skip_list: skiplist          # if skiplist is not empty
-    }
+}
 
 
 import subprocess
+
 if __name__ == "__main__":
-  print "test_params = " , test_params
-  builtin_parameters = {
-    'build_mode' : "Release",
-    'llvm_site_config' : os.path.join(os.getcwd(), 'lit.site.cfg'),
-    'clang_site_config': os.path.join(os.getcwd(), 'lit.site.cfg'),
-    'test_params' : test_params
+    print(f"test_params = {test_params}")
+    builtin_parameters = {
+        "build_mode": "Release",
+        "llvm_site_config": os.path.join(os.getcwd(), "lit.site.cfg"),
+        "clang_site_config": os.path.join(os.getcwd(), "lit.site.cfg"),
+        "test_params": test_params,
     }
-  from lit.main import main
-  main(builtin_parameters)
+    from lit.main import main
 
+    main(builtin_parameters)

ABI-Testsuite/test/basic/lit.local.cfg

@@ -1,4 +1,4 @@
 # Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 # See https://llvm.org/LICENSE.txt for license information.
 # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-config.suffixes = ['.c']
+config.suffixes = [".c"]

ABI-Testsuite/test/coll/lit.local.cfg

@@ -1,4 +1,4 @@
 # Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 # See https://llvm.org/LICENSE.txt for license information.
 # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-config.suffixes = ['.c']
+config.suffixes = [".c"]

ABI-Testsuite/test/common/lit.local.cfg

@@ -2,4 +2,4 @@
 # See https://llvm.org/LICENSE.txt for license information.
 # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 # Skip this directory
-config.suffixes = []
+config.suffixes = []