From d8bcf242c6f4170a40695e08bf88e51d49a2e435 Mon Sep 17 00:00:00 2001 From: Elyes Haouas Date: Wed, 5 Nov 2025 03:58:30 +0000 Subject: [PATCH] Revert "commonlib/endian: Silence GCC -Warray-bounds false positives" This reverts commit 668ea970757c1191f408a32fc47be7684dea2e30. Let's just keep using --param=min-pagesize=1024 in xcompile to sweep the -Warray-bounds warnings under the rug in the coreboot tree. Change-Id: I0f76c27bcbaac9d0927160fcab9cbf9aaefa9095 Signed-off-by: Elyes Haouas Reviewed-on: https://review.coreboot.org/c/coreboot/+/89915 Reviewed-by: Angel Pons Tested-by: build bot (Jenkins) --- src/commonlib/include/commonlib/endian.h | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/commonlib/include/commonlib/endian.h b/src/commonlib/include/commonlib/endian.h index 24b10826f4..971854c2b3 100644 --- a/src/commonlib/include/commonlib/endian.h +++ b/src/commonlib/include/commonlib/endian.h @@ -7,11 +7,6 @@ #include #include -/* GCC >= 12 triggers false positives (-Warray-bounds) on - * deliberate low-level memory accesses here. */ -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Warray-bounds" - /* Endian agnostic functions working on single byte. */ static inline uint8_t read_ble8(const void *src) @@ -252,4 +247,4 @@ static inline void zero_n(void *dest, size_t n) memset(dest, 0, n); } -#endif /* _COMMONLIB_ENDIAN_H_ */ +#endif