[llvm] [AMDGPU][test] fix the error case in update_mc_test_check script (PR #112731)

Ivan Kosarev via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 04:36:29 PDT 2024


================
@@ -15,15 +15,14 @@
 
 mc_LIKE_TOOLS = [
     "llvm-mc",
-    "not llvm-mc",
 ]
 ERROR_RE = re.compile(r":\d+: (warning|error): .*")
 ERROR_CHECK_RE = re.compile(r"# COM: .*")
 OUTPUT_SKIPPED_RE = re.compile(r"(.text)")
 COMMENT = {"asm": "//", "dasm": "#"}
 
 
-def invoke_tool(exe, cmd_args, testline, verbose=False):
+def invoke_tool(exe, checkRC, cmd_args, testline, verbose=False):
----------------
kosarev wrote:

Nit: `check_rc` or maybe even `check_retcode`. +Same on the caller side.

https://github.com/llvm/llvm-project/pull/112731


More information about the llvm-commits mailing list