diff options
| author | Grazvydas Ignotas <[email protected]> | 2016-04-16 04:00:10 +0300 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2016-04-25 12:23:10 +0200 |
| commit | e4fc06a2f8be2de3a08d4adfa4a26df2ae0b5cae (patch) | |
| tree | dd79c0ef687e1189bdfe3ac655e592a95180c48a /src/util | |
| parent | 787a53988cc6bb7a0f2b43c216837d683336b33f (diff) | |
util: add MAYBE_UNUSED for config dependent variables
This is mostly for variables that are only used in asserts and cause
unused-but-set-variable warnings in release builds. Could just use
UNUSED directly, but MAYBE_UNUSED should be less confusing and is
similar to what the Linux kernel has.
And yes __attribute__((unused)) can be used on variables on both GCC 4.2
(oldest supported by mesa) and clang 3.0 (just some random old version,
not sure what's the minimum for mesa).
Signed-off-by: Grazvydas Ignotas <[email protected]>
Reviewed-by: Francisco Jerez <[email protected]>
Reviewed-by: Chad Versace <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
Diffstat (limited to 'src/util')
| -rw-r--r-- | src/util/macros.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/util/macros.h b/src/util/macros.h index 0c8958feae9..f081bb82177 100644 --- a/src/util/macros.h +++ b/src/util/macros.h @@ -204,6 +204,8 @@ do { \ #define UNUSED #endif +#define MAYBE_UNUSED UNUSED + #ifdef HAVE_FUNC_ATTRIBUTE_WARN_UNUSED_RESULT #define MUST_CHECK __attribute__((warn_unused_result)) #else |