diff options
| author | Simon Marchi <simon.marchi@efficios.com> | 2024-02-21 11:46:51 -0500 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2024-02-21 13:30:17 -0500 |
| commit | aca8a74923c4a0c222a2f8f5f3e23de84ab19e77 (patch) | |
| tree | 18394829590d2ecde9cf1209178a6d5a19b7c611 | |
| parent | 99eeecc8d276e5af745e48825d66efff693a7678 (diff) | |
| download | gdb-aca8a74923c4a0c222a2f8f5f3e23de84ab19e77.zip gdb-aca8a74923c4a0c222a2f8f5f3e23de84ab19e77.tar.gz gdb-aca8a74923c4a0c222a2f8f5f3e23de84ab19e77.tar.bz2 | |
gdb: remove some GCC version checks
Since we now require C++17, and therefore gcc >= 9, it's no longer
useful to have gcc version checks for older gcc version.
Change-Id: I3a87baa03c475f2b847b422b16b69c5ff7215b54
Reviewed-by: Kevin Buettner <kevinb@redhat.com>
Approved-By: Pedro Alves <pedro@palves.net>
| -rw-r--r-- | gdb/unittests/enum-flags-selftests.c | 6 | ||||
| -rw-r--r-- | gdbserver/tracepoint.h | 6 | ||||
| -rw-r--r-- | gdbsupport/common-defs.h | 9 |
3 files changed, 1 insertions, 20 deletions
diff --git a/gdb/unittests/enum-flags-selftests.c b/gdb/unittests/enum-flags-selftests.c index 853ebca..607b8ac 100644 --- a/gdb/unittests/enum-flags-selftests.c +++ b/gdb/unittests/enum-flags-selftests.c @@ -253,10 +253,8 @@ CHECK_VALID (true, EF, true ? RE () : EF ()) CHECK_VALID (true, int, true ? EF () : EF2 ()) CHECK_VALID (true, int, true ? EF2 () : EF ()) -#if GCC_VERSION >= 5003 || defined __clang__ CHECK_VALID (true, int, true ? EF () : RE2 ()) CHECK_VALID (true, int, true ? RE2 () : EF ()) -#endif /* Same, but with an unsigned enum. */ @@ -264,10 +262,8 @@ typedef unsigned int uns; CHECK_VALID (true, uns, true ? EF () : UEF ()) CHECK_VALID (true, uns, true ? UEF () : EF ()) -#if GCC_VERSION >= 5003 || defined __clang__ CHECK_VALID (true, uns, true ? EF () : URE ()) CHECK_VALID (true, uns, true ? URE () : EF ()) -#endif /* Unfortunately this can't work due to the way C++ computes the return type of the ternary conditional operator. int isn't @@ -279,10 +275,8 @@ CHECK_VALID (true, uns, true ? URE () : EF ()) error: operands to ?: have different types ‘enum_flags<RE>’ and ‘int’ Confirmed to work with gcc 4.9, 5.3 and clang 3.7. */ -#if GCC_VERSION >= 4009 || defined __clang__ CHECK_VALID (false, void, true ? EF () : 0) CHECK_VALID (false, void, true ? 0 : EF ()) -#endif /* Check that the ++/--/<</>>/<<=/>>= operators are deleted. */ diff --git a/gdbserver/tracepoint.h b/gdbserver/tracepoint.h index 8b23232..1660c94 100644 --- a/gdbserver/tracepoint.h +++ b/gdbserver/tracepoint.h @@ -38,11 +38,7 @@ void initialize_tracepoint (void); #if defined _WIN32 || defined __CYGWIN__ # define EXPORTED_SYMBOL __declspec (dllexport) #else -# if __GNUC__ >= 4 -# define EXPORTED_SYMBOL __attribute__ ((visibility ("default"))) -# else -# define EXPORTED_SYMBOL -# endif +# define EXPORTED_SYMBOL __attribute__ ((visibility ("default"))) #endif /* Use these to make sure the functions and variables the IPA needs to diff --git a/gdbsupport/common-defs.h b/gdbsupport/common-defs.h index 8ec559e..6120719 100644 --- a/gdbsupport/common-defs.h +++ b/gdbsupport/common-defs.h @@ -187,17 +187,8 @@ #undef ATTRIBUTE_NONNULL #define ATTRIBUTE_NONNULL(m) -#if GCC_VERSION >= 3004 #define ATTRIBUTE_UNUSED_RESULT __attribute__ ((__warn_unused_result__)) -#else -#define ATTRIBUTE_UNUSED_RESULT -#endif - -#if (GCC_VERSION > 4000) #define ATTRIBUTE_USED __attribute__ ((__used__)) -#else -#define ATTRIBUTE_USED -#endif #include "libiberty.h" #include "pathmax.h" |