[llvm] r307974 - [Dominators] Define Arc less-than operator inline.

Jakub Kuderski via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 16:11:58 PDT 2017


Author: kuhar
Date: Thu Jul 13 16:11:57 2017
New Revision: 307974

URL: http://llvm.org/viewvc/llvm-project?rev=307974&view=rev
Log:
[Dominators] Define Arc less-than operator inline.

This fixes warnings on some buildbots.

Modified:
    llvm/trunk/unittests/IR/CFGBuilder.cpp
    llvm/trunk/unittests/IR/CFGBuilder.h

Modified: llvm/trunk/unittests/IR/CFGBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/IR/CFGBuilder.cpp?rev=307974&r1=307973&r2=307974&view=diff
==============================================================================
--- llvm/trunk/unittests/IR/CFGBuilder.cpp (original)
+++ llvm/trunk/unittests/IR/CFGBuilder.cpp Thu Jul 13 16:11:57 2017
@@ -16,8 +16,6 @@
 #include "llvm/Support/raw_ostream.h"
 #include "gtest/gtest.h"
 
-#include <tuple>
-
 #define DEBUG_TYPE "cfg-builder"
 
 using namespace llvm;
@@ -30,10 +28,6 @@ CFGHolder::CFGHolder(StringRef ModuleNam
 }
 CFGHolder::~CFGHolder() = default;
 
-bool llvm::operator<(const CFGBuilder::Arc &LHS, const CFGBuilder::Arc &RHS) {
-  return std::tie(LHS.From, LHS.To) < std::tie(RHS.From, RHS.To);
-}
-
 CFGBuilder::CFGBuilder(Function *F, const std::vector<Arc> &InitialArcs,
                        std::vector<Update> Updates)
     : F(F), Updates(std::move(Updates)) {

Modified: llvm/trunk/unittests/IR/CFGBuilder.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/IR/CFGBuilder.h?rev=307974&r1=307973&r2=307974&view=diff
==============================================================================
--- llvm/trunk/unittests/IR/CFGBuilder.h (original)
+++ llvm/trunk/unittests/IR/CFGBuilder.h Thu Jul 13 16:11:57 2017
@@ -23,6 +23,7 @@
 
 #include <memory>
 #include <set>
+#include <tuple>
 #include <vector>
 
 namespace llvm {
@@ -56,7 +57,10 @@ public:
     StringRef From;
     StringRef To;
 
-    friend bool operator<(const Arc &LHS, const Arc &RHS);
+    friend bool operator<(const Arc &LHS, const Arc &RHS) {
+      return std::tie(LHS.From, LHS.To) <
+             std::tie(RHS.From, RHS.To);
+    }
   };
 
   enum class ActionKind { Insert, Delete };




More information about the llvm-commits mailing list