[PATCH] D68529: [lit] Move argument parsing/validation to separate file

Julian Lettner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 7 09:54:54 PDT 2019


yln added inline comments.


================
Comment at: llvm/utils/lit/lit/main.py:237
     elif opts.incremental:
-        sort_by_incremental_cache(run)
+        run.tests.sort(key = by_mtime, reverse = True)
     else:
----------------
serge-sans-paille wrote:
> nitpicking: PEP8 recommends ``sort(key=by_mtime, reverse=True)``
Fixed. Thanks! :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68529





More information about the llvm-commits mailing list