From 0115cf6b74cc27e288678aeb7f8c977be0792be0 Mon Sep 17 00:00:00 2001 From: Christopher Haster Date: Wed, 7 May 2025 23:45:29 -0500 Subject: [PATCH] gha: Dropped explicit CFLAGS from clang testing in CI Thanks to yamt, GCC-specific flags should now be disabled if compiling with clang. Dropping the explicit flags also doubles as a test that the NO_GCC inference works. --- .github/workflows/test.yml | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index f3100bee..f18c23d4 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -454,8 +454,7 @@ jobs: TESTFLAGS="$TESTFLAGS --valgrind --context=1024 -Gdefault -Pnone" \ make test - # test that compilation is warning free under clang - # run with Clang, mostly to check for Clang-specific warnings + # compile/run with Clang, mostly to check for Clang-specific warnings test-clang: runs-on: ubuntu-latest steps: @@ -469,12 +468,8 @@ jobs: python3 --version - name: test-clang run: | - # override CFLAGS since Clang does not support -fcallgraph-info - # and -ftrack-macro-expansions - make \ - CC=clang \ - CFLAGS="$CFLAGS -MMD -g3 -I. -std=c99 -Wall -Wextra -pedantic" \ - test + CC=clang \ + make test # run benchmarks #