diff options
| author | Brox Chen <guochen2@amd.com> | 2024-10-30 09:02:40 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-30 09:02:40 -0400 |
| commit | 6f973fd4ab18ff58689e83383190ed4767c2a7dd (patch) | |
| tree | 16efa785a0fb550c57eaa59f5f878fee75a87be2 /llvm/utils/UpdateTestChecks/common.py | |
| parent | c4e135ec04a2bef5d5a5a69dfbb069a15dbf2f5e (diff) | |
| download | llvm-6f973fd4ab18ff58689e83383190ed4767c2a7dd.zip llvm-6f973fd4ab18ff58689e83383190ed4767c2a7dd.tar.gz llvm-6f973fd4ab18ff58689e83383190ed4767c2a7dd.tar.bz2 | |
[AMDGPU][test] fix the error case in update_mc_test_check script (#112731)
update_mc_test_check script handle the "error case testline" wrong in
three cases:
1. when user select "--llvm-mc-binary" with a path, the script does not
add "not" on top of the "--llvm-mc-binary" and thus getting non-zero
exit code and failed.
2. When "not" is presented in runline while not all testlines are
expected to fail, the script need to check if the "not" is needed when
it execute llvm-mc line by line. Otherwise the script will fail on
testline which is passing.
3. When there are multiple runlines, the error checkline need to use
correct line offset for "[[LINE-X]]"
This patch solve these three issues
Diffstat (limited to 'llvm/utils/UpdateTestChecks/common.py')
| -rw-r--r-- | llvm/utils/UpdateTestChecks/common.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/utils/UpdateTestChecks/common.py b/llvm/utils/UpdateTestChecks/common.py index cdfa897..b108a21 100644 --- a/llvm/utils/UpdateTestChecks/common.py +++ b/llvm/utils/UpdateTestChecks/common.py @@ -2470,6 +2470,7 @@ def get_autogennote_suffix(parser, args): "verbose", "force_update", "reset_variable_names", + "llvm_mc_binary", ): continue value = getattr(args, action.dest) |