Module: Mesa
Branch: main
Commit: dc2170f32b58345ff2d44ddef10be8c2a660e3af
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=dc2170f32b58345ff2d44ddef10be8c2a660e3af

Author: Caio Oliveira <caio.olive...@intel.com>
Date:   Mon Jan  8 10:34:22 2024 -0800

ci: Add Werror=misleading-indentation to debian-clang

Due to some issues with GCC and this warning in very long files, we
disabled it when compiling NIR.  Unfortunately by design Meson doesn't
allow us to set flags per source file.

The warning is still enabled in clang. but it is less commonly
used during development.  To avoid missing catching those warnings,
add -Werror=misleading-indentation to the GitLab CI debian-clang build.

See https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25315 for
more context.  This patch is a transcription of what Eric Engestrom
suggested, except only targetting C flags (since we only disable them
for C in NIR build).

Acked-by: Erik Faye-Lund <erik.faye-l...@collabora.com>
Reviewed-by: Eric Engestrom <e...@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26938>

---

 .gitlab-ci/build/gitlab-ci.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitlab-ci/build/gitlab-ci.yml b/.gitlab-ci/build/gitlab-ci.yml
index b7c3106f8c7..0c31760045a 100644
--- a/.gitlab-ci/build/gitlab-ci.yml
+++ b/.gitlab-ci/build/gitlab-ci.yml
@@ -469,6 +469,7 @@ debian-clang:
       -Wno-error=enum-conversion
       -Wno-error=initializer-overrides
       -Wno-error=sometimes-uninitialized
+      -Werror=misleading-indentation
     CPP_ARGS: >
       -Wno-error=c99-designator
       -Wno-error=overloaded-virtual

Reply via email to