[PATCH] D93115: LNT: Sort testsuites ascendingly

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 06:42:08 PST 2020


thopre added inline comments.


================
Comment at: lnt/server/db/v4db.py:59
+        # Order testuites alphabetically to get groupings
+        self.testsuite = dict (sorted (self.testsuite.items()))
 
----------------
Shouldn't you use an OrderedDict instead of dict to guarantee the ordering will be the same as when inserted?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93115/new/

https://reviews.llvm.org/D93115



More information about the llvm-commits mailing list