[PATCH] D51744: [WIP] Early ThinLTOLayer2 prototype

[Mandeep Singh Grang via Phabricator via llvm-commits]

mgrang added inline comments.


================
Comment at: lib/ExecutionEngine/Orc/ThinLTOLayer.cpp:362
+  // TODO Would it be good to preserve the order?
+  std::sort(JDs.begin(), JDs.end());
+  JDs.erase(std::unique(JDs.begin(), JDs.end()), JDs.end());
----------------
Please use llvm::sort instead of std::sort. See https://llvm.org/docs/CodingStandards.html#beware-of-non-deterministic-sorting-order-of-equal-elements


Repository:
  rL LLVM

https://reviews.llvm.org/D51744