[clang-tools-extra] 402baea - [modularize] Use std::tie to implement operator< (NFC) (#146220)

via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 28 13:04:04 PDT 2025


Author: Kazu Hirata
Date: 2025-06-28T13:04:00-07:00
New Revision: 402baea0a9ff7894565449e41f700c4e6a3f99cb

URL: https://github.com/llvm/llvm-project/commit/402baea0a9ff7894565449e41f700c4e6a3f99cb
DIFF: https://github.com/llvm/llvm-project/commit/402baea0a9ff7894565449e41f700c4e6a3f99cb.diff

LOG: [modularize] Use std::tie to implement operator< (NFC) (#146220)

std::tie clearly expresses the intent while slightly shortening the
code.

Added: 
    

Modified: 
    clang-tools-extra/modularize/Modularize.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/modularize/Modularize.cpp b/clang-tools-extra/modularize/Modularize.cpp
index 2a90c5e3f6782..1da531a4aefa4 100644
--- a/clang-tools-extra/modularize/Modularize.cpp
+++ b/clang-tools-extra/modularize/Modularize.cpp
@@ -459,7 +459,7 @@ struct HeaderEntry {
     return !(X == Y);
   }
   friend bool operator<(const HeaderEntry &X, const HeaderEntry &Y) {
-    return X.Loc < Y.Loc || (X.Loc == Y.Loc && X.Name < Y.Name);
+    return std::tie(X.Loc, X.Name) < std::tie(Y.Loc, Y.Name);
   }
   friend bool operator>(const HeaderEntry &X, const HeaderEntry &Y) {
     return Y < X;


        


More information about the cfe-commits mailing list