From 2941922e422057ca798d16e4a64a1a075a3dd842 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sat, 10 May 2025 09:21:24 -0700 Subject: [PATCH] [BOLT] Use std::tie to Implment operator< (NFC) --- bolt/include/bolt/Passes/FrameAnalysis.h | 5 ++--- bolt/include/bolt/Passes/PAuthGadgetScanner.h | 4 +--- bolt/include/bolt/Profile/DataReader.h | 8 +------- 3 files changed, 4 insertions(+), 13 deletions(-) diff --git a/bolt/include/bolt/Passes/FrameAnalysis.h b/bolt/include/bolt/Passes/FrameAnalysis.h index 44b54d4ed45d4..d71c338bdcc37 100644 --- a/bolt/include/bolt/Passes/FrameAnalysis.h +++ b/bolt/include/bolt/Passes/FrameAnalysis.h @@ -10,6 +10,7 @@ #define BOLT_PASSES_FRAMEANALYSIS_H #include "bolt/Passes/StackPointerTracking.h" +#include namespace llvm { namespace bolt { @@ -53,9 +54,7 @@ struct ArgInStackAccess { uint8_t Size; bool operator<(const ArgInStackAccess &RHS) const { - if (StackOffset != RHS.StackOffset) - return StackOffset < RHS.StackOffset; - return Size < RHS.Size; + return std::tie(StackOffset, Size) < std::tie(RHS.StackOffset, RHS.Size); } }; diff --git a/bolt/include/bolt/Passes/PAuthGadgetScanner.h b/bolt/include/bolt/Passes/PAuthGadgetScanner.h index ee69337637da3..66251e195822c 100644 --- a/bolt/include/bolt/Passes/PAuthGadgetScanner.h +++ b/bolt/include/bolt/Passes/PAuthGadgetScanner.h @@ -43,9 +43,7 @@ struct MCInstInBBReference { return BB == RHS.BB && BBIndex == RHS.BBIndex; } bool operator<(const MCInstInBBReference &RHS) const { - if (BB != RHS.BB) - return BB < RHS.BB; - return BBIndex < RHS.BBIndex; + return std::tie(BB, BBIndex) < std::tie(RHS.BB, RHS.BBIndex); } operator MCInst &() const { assert(BB != nullptr); diff --git a/bolt/include/bolt/Profile/DataReader.h b/bolt/include/bolt/Profile/DataReader.h index 314dcc9115586..4765eb28d53ca 100644 --- a/bolt/include/bolt/Profile/DataReader.h +++ b/bolt/include/bolt/Profile/DataReader.h @@ -90,13 +90,7 @@ struct BranchInfo { } bool operator<(const BranchInfo &RHS) const { - if (From < RHS.From) - return true; - - if (From == RHS.From) - return (To < RHS.To); - - return false; + return std::tie(From, To) < std::tie(RHS.From, RHS.To); } /// Merges branch and misprediction counts of \p BI with those of this object.