[llvm] r269637 - Revert "ThinLTO: fix non-determinism in bitcode writing"

[Mehdi Amini via llvm-commits]

Author: mehdi_amini
Date: Mon May 16 04:04:50 2016
New Revision: 269637

URL: http://llvm.org/viewvc/llvm-project?rev=269637&view=rev
Log:
Revert "ThinLTO: fix non-determinism in bitcode writing"

This reverts commit r269634, bots are broken.

From: Mehdi Amini <mehdi.amini at apple.com>

Modified:
    llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp

Modified: llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp?rev=269637&r1=269636&r2=269637&view=diff
==============================================================================
--- llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp (original)
+++ llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp Mon May 16 04:04:50 2016
@@ -3177,14 +3177,8 @@ void ModuleBitcodeWriter::writePerModule
 
   NameVals.insert(NameVals.end(), Refs.begin(), Refs.end());
 
-  std::vector<FunctionSummary::EdgeTy> Calls = FS->calls();
-  std::sort(Calls.begin(), Calls.end(),
-            [this](FunctionSummary::EdgeTy &L, FunctionSummary::EdgeTy &R) {
-              return VE.getValueID(L.first.getValue()) <
-                     VE.getValueID(R.first.getValue());
-            });
   bool HasProfileData = F.getEntryCount().hasValue();
-  for (auto &ECI : Calls) {
+  for (auto &ECI : FS->calls()) {
     NameVals.push_back(VE.getValueID(ECI.first.getValue()));
     assert(ECI.second.CallsiteCount > 0 && "Expected at least one callsite");
     NameVals.push_back(ECI.second.CallsiteCount);