no LTO for windows 2.2, impossible constraint in deneb_kzg test

mratsim · Oct 6, 2024 · aab71e3 · aab71e3
1 parent a47a753
commit aab71e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -372,12 +372,13 @@ jobs:
         # So "test_bindings" uses C and can find GMP
         # but nim-gmp cannot find GMP on Windows CI
         # Also need to workaround asynctools not being able to create pipes https://github.com/nim-lang/Nim/issues/23118
+        # And LTO impossible constraint in the deneb_kzg test (but not MSM for some reason)
         if: runner.os == 'Windows' && matrix.target.BACKEND == 'ASM'
         shell: msys2 {0}
         run: |
           cd constantine
           if [[ '${{  matrix.nim_version }}' != 'version-1-6' ]]; then
-            nimble test_no_gmp --verbose
+            nimble CTT_LTO=false test_no_gmp --verbose
           else
             nimble test_parallel_no_gmp --verbose
           fi
@@ -390,7 +391,7 @@ jobs:
         run: |
           cd constantine
           if [[ '${{  matrix.nim_version }}' != 'version-1-6' ]]; then
-            CTT_ASM=0 nimble test_no_gmp --verbose
+            CTT_ASM=0 CTT_LTO=false nimble test_no_gmp --verbose
           else
             CTT_ASM=0 nimble test_parallel_no_gmp --verbose
           fi
diff --git a/helpers/pararun.nim b/helpers/pararun.nim
@@ -113,9 +113,6 @@ proc enqueuePendingCommands(wq: WorkQueue) {.async.} =
 
     await wq.sem.acquire()
     let cmd = wq.cmdQueue.popFirst()
-    echo '\n', '-'.repeat(80)
-    echo "|\n| Scheduling #", id, "/", total, ": ", cmd ,"\n|"
-    echo '-'.repeat(80)
     let p = cmd.startProcess(options = {poStdErrToStdOut, poUsePath, poEvalCommand})
 
     let bufOut = newAsyncQueue[string]()