include: Define macro to ignore -Wdeprecated-declarations on GCC

Message ID 9df9d9e5bb4c7594b76c40613a349b4d2364e9c5.1666854355.git.research_trasio@irq.a4lg.com
State Accepted
Headers
Series include: Define macro to ignore -Wdeprecated-declarations on GCC |

Checks

Context Check Description
snail/binutils-gdb-check success Github commit url

Commit Message

Tsukasa OI Oct. 27, 2022, 7:06 a.m. UTC
  "-Wdeprecated-declarations" warning option can be helpful to track
deprecated function delarations but sometimes we need to disable this
warning for a good reason.

DIAGNOSTIC_IGNORE_DEPRECATED_DECLARATIONS is an existing macro but only
defined on Clang.  Since "-Wdeprecated-declarations" is also available on
GCC (>= 3.4.0), this commit adds equivalent definition as Clang.

__GNUC__ and __GNUC_MINOR__ are not checked because this header file seems
to assume GCC >= 4.6 (with "GCC diagnostic push/pop").

include/ChangeLog:

	* diagnostics.h (DIAGNOSTIC_IGNORE_DEPRECATED_DECLARATIONS):
	Define also on GCC.
---
 include/diagnostics.h | 3 +++
 1 file changed, 3 insertions(+)


base-commit: 2c02c72c62d2e9f590155561ebc54ca87bc10c2a
  

Comments

Alan Modra Oct. 27, 2022, 11:47 p.m. UTC | #1
On Thu, Oct 27, 2022 at 07:06:01AM +0000, Tsukasa OI via Binutils wrote:
> 	* diagnostics.h (DIAGNOSTIC_IGNORE_DEPRECATED_DECLARATIONS):
> 	Define also on GCC.

OK.
  

Patch

diff --git a/include/diagnostics.h b/include/diagnostics.h
index c1a2e8f520c..3a75f4e2719 100644
--- a/include/diagnostics.h
+++ b/include/diagnostics.h
@@ -78,6 +78,9 @@ 
 
 #elif defined (__GNUC__) /* GCC */
 
+# define DIAGNOSTIC_IGNORE_DEPRECATED_DECLARATIONS \
+  DIAGNOSTIC_IGNORE ("-Wdeprecated-declarations")
+
 # if __GNUC__ >= 7
 #  define DIAGNOSTIC_IGNORE_DEPRECATED_REGISTER \
    DIAGNOSTIC_IGNORE ("-Wregister")