From fd332753fa7050bb9d7c89147e32d285099fe402 Mon Sep 17 00:00:00 2001 From: Sergio Durigan Junior Date: Thu, 25 Oct 2018 16:46:46 -0400 Subject: Remove relational operators from common/offset-type.h This patch is a follow-up of: https://sourceware.org/ml/gdb-patches/2018-10/msg00601.html It removes the declaration of the relational operators for common/offset-type.h. As it turns out, these overloads are not being used when a new offset type is declared, because, according to Pedro Alves: I think the functions aren't called because they are templates, and thus the built-in (non-template) versions take precedence. If you make them non-templates, then they should be called. But, the built-ins are fine, so yeah, we can just remove the custom definitions. The patch also adjusts the comments on the code. No regressions introduced. gdb/ChangeLog: 2018-10-29 Sergio Durigan Junior * common/offset-type.h (DEFINE_OFFSET_REL_OP): Delete. Adjust comments. --- gdb/common/offset-type.h | 18 +----------------- 1 file changed, 1 insertion(+), 17 deletions(-) (limited to 'gdb/common') diff --git a/gdb/common/offset-type.h b/gdb/common/offset-type.h index b480b14..174ad1e 100644 --- a/gdb/common/offset-type.h +++ b/gdb/common/offset-type.h @@ -57,7 +57,7 @@ /* The macro macro is all you need to know use offset types. The rest below is all implementation detail. */ -/* For each enum class type that you want to support relational +/* For each enum class type that you want to support arithmetic operators, declare an "is_offset_type" overload that has exactly one parameter, of type that enum class. E.g.,: @@ -73,22 +73,6 @@ function via ADL. */ -#define DEFINE_OFFSET_REL_OP(OP) \ - template ()))> \ - constexpr bool \ - operator OP (E lhs, E rhs) \ - { \ - using underlying = typename std::underlying_type::type; \ - return (static_cast (lhs) \ - OP static_cast (lhs)); \ - } - -DEFINE_OFFSET_REL_OP(>) -DEFINE_OFFSET_REL_OP(>=) -DEFINE_OFFSET_REL_OP(<) -DEFINE_OFFSET_REL_OP(<=) - /* Adding or subtracting an integer to an offset type shifts the offset. This is like "PTR = PTR + INT" and "PTR += INT". */ -- cgit v1.1