From 9c3967db0f92b2a44ea4780a32897f4abbef4e3c Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Mon, 6 Jul 2015 19:24:40 +0000 Subject: [PATCH] Swap operands instead of using !. This avoids returning true for A == B. Thanks to Benjamin Kramer for noticing it. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@241490 91177308-0d34-0410-b5e6-96231b3b80d8 --- tools/llvm-nm/llvm-nm.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/llvm-nm/llvm-nm.cpp b/tools/llvm-nm/llvm-nm.cpp index c30d628d9a5..b750b5f755a 100644 --- a/tools/llvm-nm/llvm-nm.cpp +++ b/tools/llvm-nm/llvm-nm.cpp @@ -510,7 +510,7 @@ static void sortAndPrintSymbolList(SymbolicFile &Obj, bool printName, Cmp = compareSymbolName; if (ReverseSort) - Cmp = [=](const NMSymbol &A, const NMSymbol &B) { return !Cmp(A, B); }; + Cmp = [=](const NMSymbol &A, const NMSymbol &B) { return Cmp(B, A); }; std::sort(SymbolList.begin(), SymbolList.end(), Cmp); } -- 2.34.1