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

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 19 23:26:54 PDT 2018


mgrang added inline comments.


================
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;
----------------
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/D43962





More information about the llvm-commits mailing list