Skip to content

Commit a9b76dc

Browse files
thugheskartben
authored andcommitted
include: Re-enable warning
This warning no longer seems to be triggering, so we shouldn't disable it. Signed-off-by: Tom Hughes <[email protected]>
1 parent 84d4c8a commit a9b76dc

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

include/zephyr/sys/cbprintf_internal.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -722,17 +722,6 @@ do { \
722722
#define Z_CBPRINTF_PACK_ARG(arg_idx, arg) \
723723
Z_CBPRINTF_PACK_ARG2(arg_idx, _pbuf, _pkg_len, _pkg_offset, _pmax, arg)
724724

725-
/* When using clang additional warning needs to be suppressed since each
726-
* argument of fmt string is used for sizeof() which results in the warning
727-
* if argument is a string literal. Suppression is added here instead of
728-
* the macro which generates the warning to not slow down the compiler.
729-
*/
730-
#ifdef __clang__
731-
#define Z_CBPRINTF_SUPPRESS_SIZEOF_ARRAY_DECAY \
732-
_Pragma("GCC diagnostic ignored \"-Wsizeof-array-decay\"")
733-
#else
734-
#define Z_CBPRINTF_SUPPRESS_SIZEOF_ARRAY_DECAY
735-
#endif
736725

737726
/* Allocation to avoid using VLA and alloca. Alloc frees space when leaving
738727
* a function which can lead to increased stack usage if logging is used
@@ -780,7 +769,6 @@ do { \
780769
do { \
781770
_Pragma("GCC diagnostic push") \
782771
_Pragma("GCC diagnostic ignored \"-Wpointer-arith\"") \
783-
Z_CBPRINTF_SUPPRESS_SIZEOF_ARRAY_DECAY \
784772
BUILD_ASSERT(!IS_ENABLED(CONFIG_XTENSA) || \
785773
(IS_ENABLED(CONFIG_XTENSA) && \
786774
!((_align_offset) % CBPRINTF_PACKAGE_ALIGNMENT)), \

0 commit comments

Comments
 (0)