Skip to content

Commit 3c9b316

Browse files
author
Mandeep Singh Grang
authored
Fix calls to ResultCode after clang 11 upgrade (#105)
After upgrading checkedc-clang to release 11, we see that the signature of ResultCode in lit has changed and it now expects 3 parameters. But our checkedc-llvm-test-suite has not been updated and continues to pass only 2 parameters in calls to ResultCode. This causes the following error in the Linux LNT Benchmarking ADO run: "TypeError: __new__() takes exactly 4 arguments (3 given)" Here we are making/fixing calls to ResultCode similar to those in https://github.com/microsoft/checkedc-clang/blob/master/llvm/utils/lit/lit/Test.py#L45 The long term fix is to upgrade the checkedc-llvm-test-suite.
1 parent 4b2f018 commit 3c9b316

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

litsupport/test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
import os
1414

1515

16-
SKIPPED = lit.Test.ResultCode('SKIPPED', False)
17-
NOEXE = lit.Test.ResultCode('NOEXE', True)
16+
SKIPPED = lit.Test.ResultCode('SKIPPED', 'Skipped', False)
17+
NOEXE = lit.Test.ResultCode('NOEXE', 'NoExe', True)
1818

1919

2020
class TestSuiteTest(lit.formats.ShTest):

0 commit comments

Comments
 (0)