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

Alexander Richardson via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 15:56:08 PDT 2024


================
@@ -97,8 +109,16 @@ def getStdCheckLine(prefix, output, mc_mode):
     return o
 
 
-def getErrCheckLine(prefix, output, mc_mode):
-    return COMMENT[mc_mode] + " " + prefix + ": " + ":[[@LINE-1]]" + output + "\n"
+def getErrCheckLine(prefix, output, mc_mode, line_offset=1):
+    return (
+        COMMENT[mc_mode]
+        + " "
+        + prefix
+        + ": "
+        + ":[[@LINE-{}]]".format(line_offset)
----------------
arichardson wrote:

I agree an f-string would make this easier to read.

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


More information about the llvm-commits mailing list