figuring out cause of error

skipping ctest, checking if integration tests pass
lcompilers · Jul 23, 2024 · 388e640 · 388e640
1 parent 17dca10
commit 388e640
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 54 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -540,7 +540,7 @@ jobs:
       - name: Test Linux LLVM ${{ matrix.llvm-version }}
         shell: bash -e -l {0}
         run: |
-            ctest --output-on-failure
+            # ctest --output-on-failure
             cd integration_tests
             ./run_tests.py -b llvm llvm_jit
             ./run_tests.py -b llvm llvm_jit -f

diff --git a/src/lpython/tests/test_llvm.cpp b/src/lpython/tests/test_llvm.cpp
@@ -41,59 +41,59 @@ define i64 @f2()
     //CHECK(e.execfn<int64_t>("f2") == 5);
 }
 
-// TEST_CASE("llvm 1 fail") {
-//     LCompilers::LLVMEvaluator e;
-//     CHECK_THROWS_AS(e.add_module(R"""(
-// define i64 @f1()
-// {
-//     ; FAIL: "=x" is incorrect syntax
-//     %1 =x alloca i64
-// }
-//         )"""), LCompilers::LCompilersException);
-//     CHECK_THROWS_WITH(e.add_module(R"""(
-// define i64 @f1()
-// {
-//     ; FAIL: "=x" is incorrect syntax
-//     %1 =x alloca i64
-// }
-//         )"""), "parse_module(): Invalid LLVM IR");
-// }
-
-
-// TEST_CASE("llvm 2") {
-//     LCompilers::LLVMEvaluator e;
-//     e.add_module(R"""(
-// @count = global i64 0
-
-// define i64 @f1()
-// {
-//     store i64 4, i64* @count
-//     %1 = load i64, i64* @count
-//     ret i64 %1
-// }
-//     )""");
-//     CHECK(e.execfn<int64_t>("f1") == 4);
-
-//     e.add_module(R"""(
-// @count = external global i64
-
-// define i64 @f2()
-// {
-//     %1 = load i64, i64* @count
-//     ret i64 %1
-// }
-//     )""");
-//     CHECK(e.execfn<int64_t>("f2") == 4);
-
-//     CHECK_THROWS_AS(e.add_module(R"""(
-// define i64 @f3()
-// {
-//     ; FAIL: @count is not defined
-//     %1 = load i64, i64* @count
-//     ret i64 %1
-// }
-//         )"""), LCompilers::LCompilersException);
-// }
+TEST_CASE("llvm 1 fail") {
+    LCompilers::LLVMEvaluator e;
+    CHECK_THROWS_AS(e.add_module(R"""(
+define i64 @f1()
+{
+    ; FAIL: "=x" is incorrect syntax
+    %1 =x alloca i64
+}
+        )"""), LCompilers::LCompilersException);
+    CHECK_THROWS_WITH(e.add_module(R"""(
+define i64 @f1()
+{
+    ; FAIL: "=x" is incorrect syntax
+    %1 =x alloca i64
+}
+        )"""), "parse_module(): Invalid LLVM IR");
+}
+
+
+TEST_CASE("llvm 2") {
+    LCompilers::LLVMEvaluator e;
+    e.add_module(R"""(
+@count = global i64 0
+
+define i64 @f1()
+{
+    store i64 4, i64* @count
+    %1 = load i64, i64* @count
+    ret i64 %1
+}
+    )""");
+    CHECK(e.execfn<int64_t>("f1") == 4);
+
+    e.add_module(R"""(
+@count = external global i64
+
+define i64 @f2()
+{
+    %1 = load i64, i64* @count
+    ret i64 %1
+}
+    )""");
+    CHECK(e.execfn<int64_t>("f2") == 4);
+
+    CHECK_THROWS_AS(e.add_module(R"""(
+define i64 @f3()
+{
+    ; FAIL: @count is not defined
+    %1 = load i64, i64* @count
+    ret i64 %1
+}
+        )"""), LCompilers::LCompilersException);
+}
 
 TEST_CASE("llvm 3") {
     LCompilers::LLVMEvaluator e;