Remove relational operators from common/offset-type.h
authorSergio Durigan Junior <sergiodj@redhat.com>
Thu, 25 Oct 2018 20:46:46 +0000 (16:46 -0400)
committerSergio Durigan Junior <sergiodj@redhat.com>
Tue, 30 Oct 2018 03:48:37 +0000 (23:48 -0400)
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  <sergiodj@redhat.com>

* common/offset-type.h (DEFINE_OFFSET_REL_OP): Delete.
Adjust comments.

gdb/ChangeLog
gdb/common/offset-type.h

index 51ba04d..0207bec 100644 (file)
@@ -1,3 +1,8 @@
+2018-10-29  Sergio Durigan Junior  <sergiodj@redhat.com>
+
+       * common/offset-type.h (DEFINE_OFFSET_REL_OP): Delete.
+       Adjust comments.
+
 2018-10-29  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
        * procfs.c: Include common/pathstuff.h.
index b480b14..174ad1e 100644 (file)
@@ -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.,:
 
    function via ADL.
 */
 
-#define DEFINE_OFFSET_REL_OP(OP)                                       \
-  template<typename E,                                                 \
-          typename = decltype (is_offset_type (std::declval<E> ()))>   \
-  constexpr bool                                                       \
-  operator OP (E lhs, E rhs)                                           \
-  {                                                                    \
-    using underlying = typename std::underlying_type<E>::type;         \
-    return (static_cast<underlying> (lhs)                              \
-           OP static_cast<underlying> (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".  */