[PATCH] D97098: [Utils] Add an option to specify number of cores to use in creduce-clang-crash.py

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 22 11:14:02 PST 2021


rnk added inline comments.


================
Comment at: clang/utils/creduce-clang-crash.py:81
     self.creduce_flags = ["--tidy"]
+    if core_number > 0:
+      self.creduce_flags = ["--n", str(core_number)]
----------------
I think we should try to pick a good default here. I think we can do better than creduce's default of 4 or whatever it is. I think `max(4, os.cpu_count() / 2)` might be a good guess.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97098



More information about the cfe-commits mailing list