[Lldb-commits] [lldb] [lldb] Use std::tie to implement operator< (NFC) (PR #141416)

[Kazu Hirata via lldb-commits]

https://github.com/kazutakahirata created 
https://github.com/llvm/llvm-project/pull/141416

None

>From e2a76ae23d253dc1802d2eb4fb3fc2789b5ee64f Mon Sep 17 00:00:00 2001
From: Kazu Hirata <k...@google.com>
Date: Sat, 24 May 2025 20:51:21 -0700
Subject: [PATCH] [lldb] Use std::tie to implement operator< (NFC)

---
 lldb/include/lldb/Symbol/CompilerDeclContext.h       |  5 ++---
 lldb/include/lldb/Target/CoreFileMemoryRanges.h      |  7 ++-----
 .../LanguageRuntime/ObjC/ObjCLanguageRuntime.h       | 12 ++----------
 3 files changed, 6 insertions(+), 18 deletions(-)

diff --git a/lldb/include/lldb/Symbol/CompilerDeclContext.h 
b/lldb/include/lldb/Symbol/CompilerDeclContext.h
index 89b4a9787688b..fe2ef215a8d19 100644
--- a/lldb/include/lldb/Symbol/CompilerDeclContext.h
+++ b/lldb/include/lldb/Symbol/CompilerDeclContext.h
@@ -48,9 +48,8 @@ class CompilerDeclContext {
   explicit operator bool() const { return IsValid(); }
 
   bool operator<(const CompilerDeclContext &rhs) const {
-    if (m_type_system == rhs.m_type_system)
-      return m_opaque_decl_ctx < rhs.m_opaque_decl_ctx;
-    return m_type_system < rhs.m_type_system;
+    return std::tie(m_type_system, m_opaque_decl_ctx) <
+           std::tie(rhs.m_type_system, rhs.m_opaque_decl_ctx);
   }
 
   bool IsValid() const {
diff --git a/lldb/include/lldb/Target/CoreFileMemoryRanges.h 
b/lldb/include/lldb/Target/CoreFileMemoryRanges.h
index 0cc5433525ddc..78d01acca324e 100644
--- a/lldb/include/lldb/Target/CoreFileMemoryRanges.h
+++ b/lldb/include/lldb/Target/CoreFileMemoryRanges.h
@@ -30,11 +30,8 @@ struct CoreFileMemoryRange {
   }
 
   bool operator<(const CoreFileMemoryRange &rhs) const {
-    if (range < rhs.range)
-      return true;
-    if (range == rhs.range)
-      return lldb_permissions < rhs.lldb_permissions;
-    return false;
+    return std::tie(range, lldb_permissions) <
+           std::tie(rhs.range, rhs.lldb_permissions);
   }
 
   std::string Dump() const {
diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.h 
b/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.h
index ffe9725fa6826..45de098c15f51 100644
--- a/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.h
+++ b/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.h
@@ -386,16 +386,8 @@ class ObjCLanguageRuntime : public LanguageRuntime {
     }
 
     bool operator<(const ClassAndSel &rhs) const {
-      if (class_addr < rhs.class_addr)
-        return true;
-      else if (class_addr > rhs.class_addr)
-        return false;
-      else {
-        if (sel_addr < rhs.sel_addr)
-          return true;
-        else
-          return false;
-      }
+      return std::tie(class_addr, sel_addr) <
+             std::tie(rhs.class_addr, rhs.sel_addr);
     }
 
     lldb::addr_t class_addr = LLDB_INVALID_ADDRESS;

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits