[PATCH] D43962: [GlobalISel][utils] Adding the init version of Instruction Select Testgen

Roman Tereshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 25 10:42:18 PDT 2018


rtereshin added a comment.

ping



================
Comment at: lib/CodeGen/GlobalISel/InstructionSelectorTestgen.cpp:260
+      Keys.push_back(Item.first);
+    std::sort(Keys.begin(), Keys.end(), std::greater<decltype(Keys.front())>());
+    BitVector VisitedPhysRegs;
----------------
mgrang wrote:
> Please use llvm::sort instead of std::sort. See https://llvm.org/docs/CodingStandards.html#beware-of-non-deterministic-sorting-order-of-equal-elements.
Good catch, thanks, will do!


Repository:
  rL LLVM

https://reviews.llvm.org/D43962





More information about the llvm-commits mailing list