[PATCH] D138953: llvm-reduce: Try to kill parallel workitems once we have a result.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 14:31:29 PST 2022


arsenm created this revision.
arsenm added reviewers: Meinersbur, fhahn, regehr, lebedev.ri, aeubanks.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

The current reduction logic tries to reproduce what a serial reduction
would produce, and just takes the first one that is still
interesting. We still have to wait for all others to complete though,
which at that point is just a waste.

      

This helps speed things up with long running reducers, which I
frequently have. e.g. for the added sleep test on my system, it took
about 8 seconds before this change and about 4 after.


https://reviews.llvm.org/D138953

Files:
  llvm/test/tools/llvm-reduce/Inputs/sleep.py
  llvm/test/tools/llvm-reduce/parallel-workitem-kill.ll
  llvm/tools/llvm-reduce/ReducerWorkItem.h
  llvm/tools/llvm-reduce/TestRunner.cpp
  llvm/tools/llvm-reduce/TestRunner.h
  llvm/tools/llvm-reduce/deltas/Delta.cpp
  llvm/tools/llvm-reduce/llvm-reduce.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138953.478726.patch
Type: text/x-patch
Size: 12115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221129/192b33c9/attachment.bin>


More information about the llvm-commits mailing list