Removed redundant Valgrind tests

I think we don't need to test both -O2 and -O3. We're checking memory
error here, they should not be much different under different
optimisation settings (though maybe -O3 can trigger more bugs).

Also did not add myself to the list of authors, I think this trivial
change is not copyrightable.

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index a6182ab4301f35726567350e51436627b2b92a41..74ba013563f91813f060d6b0c87a0d45bc0bc09e 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -148,14 +148,6 @@ jobs:
       - name: Build, then run with valgrind (OK without march=native)
         run: |
 
-          echo "Valgrind on GCC build with O2"
-          make clean; make test.out CFLAGS="-O2" -j4
-          valgrind ./test.out
-
-          echo "Valgrind on CLANG build with O2"
-          make clean; make test.out CC="clang -std=c99" CFLAGS="-O2" -j4
-          valgrind ./test.out
-
           echo "Valgrind on GCC build with O3"
           make clean; make test.out CFLAGS="-O3" -j4
           valgrind ./test.out